Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Preparation for Debian inclusion #21

Open
wants to merge 9 commits into
base: without_jganalytics
Choose a base branch
from

Conversation

adiknoth
Copy link
Contributor

Hi!

Here's a series of patches I've made while working on the Debian package.

For the sake of simplicity, I've moved the build.xml to the toplevel directory.

I'll file a ticket regarding runtime support, that is, running graphtea from any directory.

Changes:

   - creates binary/ and build/ inside the toplevel directory
   - fix clean target
   - fix forgotten deltree statement outside any target
Replaced by toplevel build.xml
With the latest changes, graphtea can be run from any directory, so it's
no longer necessary to cd into binary/.
Those files will be provided by Debian in /usr/share/java/
@adiknoth
Copy link
Contributor Author

I think it's OK to pull this into your without_jganalytics branch by merging the pull request, however, we don't really need this branch. I'll send you another pull request for master.

Debian will then create its own upstream branch by excluding all the precompiled files.

@azinazadi
Copy link
Contributor

Hello Dear @adiknoth ,

Do you still want to work on this? that would be very nice. If so, I will apply all the changes on GraphTea.

please let me know :)
thx,
Azin

@adiknoth
Copy link
Contributor Author

Hi!

Ever since I've left FSU Jena, I no longer have time to work on that. Just ask Ali if he could find a student to do it.

Cheers

@azinazadi
Copy link
Contributor

Hello Dear @adiknoth

thanks. I will ask him. If you tell me how to do it or send a guideline I can also work on it...

best,
Azin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants