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

Update setup.py and remove useless files #8

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

artzet-s
Copy link
Contributor

@artzet-s artzet-s commented Apr 2, 2020

No description provided.

@pradal
Copy link
Contributor

pradal commented Apr 2, 2020

There was a bug.
The package has been restructured, but the old packages at the root have not all been moved.

Either you move the packages into openalea, or you let openalea manage the namespace.
For instance, both
stocatree and openalea/stoctree exists. But also other packages at the src/

@artzet-s
Copy link
Contributor Author

artzet-s commented Apr 2, 2020

Hi @pradal , thanks for review !

Git doesn't delete folder if you checkout a version, but it will delete all empty folder on the remote repository (is for that that we cannot push empty folder).

If your look at my fork (https://github.com/artzet-s/MAppleT) I don't have anymore stocatree in src at least.

I not moved cpp folder because I don't know why it's here in fact ... It seems don't be call (Makefile have static path of /home/cokelaer/.. ) so I don't know what to do with that and I don't move it. I think it serve as script/routine before .

Same for wralea, I don't knew if I have to move them as well. But I understand now and I will move them into openalea dir.

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