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

Remove/update deprecated files #788

Closed
durack1 opened this issue Oct 10, 2014 · 23 comments
Closed

Remove/update deprecated files #788

durack1 opened this issue Oct 10, 2014 · 23 comments
Assignees
Milestone

Comments

@durack1
Copy link
Member

durack1 commented Oct 10, 2014

I've just pulled down the 2.0.0 source and noted the files TODO.txt and Changes.txt which don't appear to be relevant to the latest master repo - should these be removed/updated?

@durack1
Copy link
Member Author

durack1 commented Oct 10, 2014

Same goes for the tests directory, it appears to contain cdat tests which are replicated in the testing subdir?

@durack1
Copy link
Member Author

durack1 commented Oct 10, 2014

Is the libcdms directory also obsolete - has this source migrated into the Packages subdir?

@aashish24
Copy link
Contributor

  • I think we should get rid of TODO.txt and update the Changes.txt.
  • @doutriaux1 can we remove the tests directory?
  • Yes, I think libcdms is moved under Packages..

@durack1
Copy link
Member Author

durack1 commented Oct 10, 2014

The docs directory also contains a bunch of stuff that looks about 5+ years old.. The release notes nicely document some changes with cdat 4.3..!

@durack1
Copy link
Member Author

durack1 commented Oct 10, 2014

I noted that a "release notes" link appeared (and has now disappeared) against the 2.0.0 tag under the releases page - is there a way to get github to autogenerate this from commits?

@durack1
Copy link
Member Author

durack1 commented Nov 24, 2014

@doutriaux1 @aashish24 can you mark this as a 2.2 milestone please?

@doutriaux1 doutriaux1 added this to the 2.2 milestone Nov 24, 2014
@durack1
Copy link
Member Author

durack1 commented Jan 7, 2015

As noted in PCMDI/pcmdi_metrics#161 here is a more specific listing of files/dirs to cleanup:

  • Cleanup deprecated subdirs/files during install: bin/{.gif, .ttf, .png, .scr, .xmb}, bin/images, bin/tutorials, include, man, share/man

@durack1
Copy link
Member Author

durack1 commented Feb 19, 2015

@doutriaux1 you should probably push this to a 2.3 milestone..

@durack1
Copy link
Member Author

durack1 commented May 28, 2015

This issue is related to #1353 - @doutriaux1 will need to complete the merging of images and resources subdirs and their install locations..

durack1 added a commit that referenced this issue Jun 15, 2015
@durack1 durack1 modified the milestones: 3.0, 2.4 Sep 30, 2015
@durack1
Copy link
Member Author

durack1 commented Sep 30, 2015

@doutriaux1 @aashish24 it seems my PR #1341 was closed, so this should no longer be a 2.4.0 milestone - I've amended this to 3.0 it would be nice to get this cleanup underway..

@doutriaux1
Copy link
Contributor

@durack1 I will produce a fresh new PR when I do this.

@durack1 durack1 modified the milestones: 2.6, 3.0 Jan 4, 2016
@durack1
Copy link
Member Author

durack1 commented Jan 4, 2016

@doutriaux1 2.6 would be a good time to get this cleanup complete..

@doutriaux1
Copy link
Contributor

I think this is irrelevent with conda

@doutriaux1 doutriaux1 modified the milestones: 3.0, 2.6 May 25, 2016
@durack1
Copy link
Member Author

durack1 commented May 25, 2016

@doutriaux1 I don't.. if it's dead, then kill it and clean things up so that redundant and many year old rubbish isn't polluting a clean repo..

@doutriaux1
Copy link
Contributor

@durack1 it's kind of my point when I switch to the conda build, all this is going to didsappear from the repo

@durack1
Copy link
Member Author

durack1 commented May 25, 2016

@doutriaux1 that is the conda variants.. But not the source, by which some folks will still persistently try to build from..

@aashish24
Copy link
Contributor

I agree with @durack1. Build system and this issue is orthogonal things. In my rst_docs branch I took care of the docs folder but we still lot of files un-used and not maintained. I strongly suggest we remove them or update them.

@durack1
Copy link
Member Author

durack1 commented May 26, 2016

@doutriaux1 @aashish24 FYI there is a closed PR that I submitted many moons ago (a year ago) #1341 - it was closed without merging

@doutriaux1
Copy link
Contributor

@aashish24 when updating fully to conda that will imply doing the cleaning. No worries. Should be done before 2.6, which is when we officially switch to conda for releasing.

@aashish24
Copy link
Contributor

@doutriaux1 I think we are mixing issues here. Conda or not conda should be a different issue. We should not solve every problem with one bullet 😄. These two issues are orthogonal and it would be better if we have two separate PR's for them.

@doutriaux1
Copy link
Contributor

not really, lots of these files are not necessary really but are used by the insane install.py.in that will go away with conda build.

@durack1
Copy link
Member Author

durack1 commented May 26, 2016

@doutriaux1 @aashish24 I see overlap with these standing issues:
#788 - this issue
#1353 - Merge contents of images and resources subdirs (this is closed, but shouldn't really be)
#1354 - Update license/legal info

@durack1
Copy link
Member Author

durack1 commented Jul 5, 2016

@doutriaux1 you have sorted this by migrating to conda, closing

@durack1 durack1 closed this as completed Jul 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants