-
Notifications
You must be signed in to change notification settings - Fork 146
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
Branch cleanup #1
Comments
The last commit for the branches directory in SVN (20322) says: "removed branches/Xie_Zhinan, which was unused". Does that mean we can remove the Xie_Zhinan branch from git entirely? |
Ah, I have figured out why some commits appear in triplicate. A long time ago, some development was done in a subdirectory, which eventually replaced the top-level directory. Because of this, git followed development back through history and ended up with commits on |
new_branch_for_Xie_Zhinan: This is a completely disjoint branch containing 56 commits which are all empty. @eheien: did something go wrong with the conversion here, or did you just remove all the directories affected by these commits? It looks like most (all?) of the commits on this branch were in the examples directory, which I removed and put on the CIG servers for space reasons. So I don't believe anything went wrong, the changes were just made in a directory no longer in the Github repository. |
For future reference, you can use the |
Thanks for the tip. On Nov 13, 2013, at 4:23 PM, Jed Brown notifications@github.com wrote:
|
Hi, That branch was created by mistake and never used, you can safely delete Thanks, On 11/14/2013 01:16 AM, Eric Heien wrote:
Dimitri Komatitsch |
Dear Elliott, Since David Luet and Eric Heien are also cleaning the official repo on Thanks, On 11/14/2013 12:50 AM, Elliott Sales de Andrade wrote:
Dimitri Komatitsch |
I actually started this yesterday and finished it already. It's available at https://github.com/QuLogic/specfem2d-clean as soon as it finishes uploading. The far history was pretty messy, but it turns out that the triplicate commits are all redundant (I verified this by comparing all commits that were duplicated). I just dropped the split section, and that saved about 400 commits. This cleanup drops the number of commits from 1517 to just shy of 1000 and the .git size from ~106M to ~37M, which is a nice savings. |
Thanks for working on this Elliott. If it’s OK with David and the rest of the SPECFEM team I’d be fine with replacing the existing SPECFEM2D/3D Github repositories with your changes. If there are any problems, at worst we can revert to the original based on the current SVN repository. Does anyone have any concerns about this? On Nov 13, 2013, at 9:42 PM, Elliott Sales de Andrade notifications@github.com wrote:
|
Hi Eric and Jed, Yes, please just delete that, because all those early empty comments from Thank you so much. Best regards, 2013/11/14 Eric Heien notifications@github.com
|
Fine with me. Dimitri. On 11/14/2013 06:53 AM, Eric Heien wrote:
Dimitri Komatitsch |
If Dimitri is ok with it then that's what we should do. On Thu, Nov 14, 2013 at 5:57 AM, komatits notifications@github.com wrote:
David Luet |
Only the code will be changed, the changes to issues, wiki, etc. will remain untouched. I’ll go ahead and bring Elliott’s repositories over today. -Eric On Nov 14, 2013, at 10:10 AM, David Luet notifications@github.com wrote:
|
I just copied the SPECFEM2D and 3D clean repositories from Elliott into the official CIG organization repos at: https://github.com/geodynamics/specfem2d Please take a look at them and let us know if there are any problems. I still have copies of the original repositories so we can restore them quickly if there are any problems. Thanks to Elliott for doing the hard work of cleaning everything up. On Nov 13, 2013, at 9:42 PM, Elliott Sales de Andrade notifications@github.com wrote:
|
Looks good to me; the refs all seem to match what I have. We will need to send a message to the mailing list for everyone to recreate all their forks and clones. |
This repo's cleanup looks complete. |
There seem to be several extra branches that should be deleted or merged somehow:
These are merged and irrelevant. They should be deleted:
The text was updated successfully, but these errors were encountered: