<div>Hi!</div><div> </div><div> </div><div>} <span style="background-color:#ffffff;color:#000000;float:none;font-family:'ys text' , 'arial' , sans-serif;font-size:16px;font-style:normal;font-weight:400;text-decoration-style:initial;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">Were there any failing tests ( VVM: "test and(or) source code from examples folder and(or) s.c. of GUI utilities" ) that the reverts fixed?</span></div><div> </div><div>Yes, of course.  I sent results of testing on Github in comments of related commit.</div><div> </div><div>Best regards, Victor Miasnikov</div><div> </div><div> </div><div>26.10.2021, 04:36, "Rodney Bates" <rodney.bates@gmx.com>:</div><blockquote><p><br /><br />On 10/25/21 6:42 AM, Eric Sessoms wrote:</p><blockquote> Hi all,<br /><br /> When I reverted the WIP on constructors, I made a point to preserve that work on the branch rodney-was-here, and I took care that branch had enough shared history with master that it would merge cleanly and the new constructor work would take precedence over the revert when it did get merged in.<br /><br /> I had noticed that some committers (not just Rodney) seem to struggle with git, so I was making sure it would be as easy as possible to undo.<br /><br /> If you tried to pull with uncommitted work, git would have refused unless you stashed it.  That stash will show if you type:<br /><br /> git stash list<br /><br /> You can resurrect it with "git stash apply", OR, because stashes can float between branches, you can checkout the branch rodney-was-here and *then* type "git stash apply" and you should be exactly where you were before my revert.<br /><br /> Of course, if your goal is to undo the revert, it can be reverted like any other commit.  The nuclear option is is to reset the repository to before my revert:<br /><br /> git reset dee76c387ab3cd226851ad2ef9f11a71f85cd55d<br /> git push --force origin<br /><br /> But for that to work you need to disable branch protection on master here <a href="https://github.com/modula3/cm3/settings/branches" rel="noopener noreferrer">https://github.com/modula3/cm3/settings/branches</a> Warning: "reset" will destroy your WIP if it is not saved elsewhere.<br /><br /> That gives you at least four different ways to undo my undo (merge, stash, revert, and reset).<br /><br /> Of course I was aware that "new guy" reverting work by a senior dev would be a touchy issue.  I let HEAD sit broken for two full weeks before taking any action.  And I wasn't quiet about it.  So this revert was also a question: is this project trying to produce a working Modula-3 compiler, or is it more of a playground sort of thing?<br /><br /> I think it's great that Rodney is trying to fix this longstanding and obviously very difficult bug, but there's no technical reason the whole rest of the world has to sit broken while he figures it out.  Our tools--git in particular--are better than that.</blockquote><p><br />Eric, I am not complaining about the reverts, just trying to get my fixes back.<br />And I do appreciate the work you are doing.<br /><br /> From my local testing on my one target (AMD64_LINUX), before and<br />since, I believe the two commits in question did not introduce any new bugs.<br />failures that have shown up since have been either prexisting bugs that were<br />uncovered by the fixes or previously undiscovered, or problems with the tests,<br />not the compiler.  Were there any failing tests that the reverts fixed?<br /><br />I will be more cautious about using branches.  But it is scary. Somewhere<br />in my git book, there is a statement that checkout is dangerous (can't locate<br />it now), which you have to do to switch to a branch.  OTOH, I find this<br />statement: "if your working directory or staging area has uncommitted<br />changes that conflict with the branch you're checking out, Git won't let<br />you switch branches".  (ProGit, p51).  It did not happen that way for me.<br />I have not used "--force" option anywhere.<br /><br />Some of the reverted files had additional changes since the earlier commits,<br />so just undoing the revert, I would expect not to get me back. However,<br />I have been looking into automatic daily snapshots that I have set up, and<br />it appears one contains what I want.  So that is another possible way.<br /> </p><blockquote><br /> But if a working compiler is not the primary goal, then I'm sorry to have bothered you, and I will f*** right on off.<br /><br /><br /> On 10/25/21 6:27 AM, <a href="mailto:vvm@tut.by" rel="noopener noreferrer">vvm@tut.by</a> wrote:<blockquote> + coder5506@<br /> Hi!<br /> (<br />   I can prepare in GITKraken needed "pool requests"<br /> )<br /> 2)<br /> } the changes in the former two are now gone, neither in either branch<br /> Are You see this branch?<br /> <a href="https://github.com/modula3/cm3/commits/rodney-was-here" rel="noopener noreferrer">https://github.com/modula3/cm3/commits/rodney-was-here</a> <<a href="https://github.com/modula3/cm3/commits/rodney-was-here" rel="noopener noreferrer">https://github.com/modula3/cm3/commits/rodney-was-here</a>><br /> 1)<br /> Which variant<br /> remoteConsructors<br /> (               ^^^^ without char "t"   )<br /> remoteConsructors<br /> is good?<br /> I.e. this<br /> <a href="https://github.com/modula3/cm3/commits/remoteConsructors" rel="noopener noreferrer">https://github.com/modula3/cm3/commits/remoteConsructors</a> <<a href="https://github.com/modula3/cm3/commits/remoteConsructors" rel="noopener noreferrer">https://github.com/modula3/cm3/commits/remoteConsructors</a>><br />   or this<br /> <a href="https://github.com/modula3/cm3/commits/remoteConstructors" rel="noopener noreferrer">https://github.com/modula3/cm3/commits/remoteConstructors</a> <<a href="https://github.com/modula3/cm3/commits/remoteConstructors" rel="noopener noreferrer">https://github.com/modula3/cm3/commits/remoteConstructors</a>><br /> ?<br /> Are You see ( as I see in GUI GIT software GITKraken ) this?<br /> Best regards, Victor Miasnikov<br /> 24.10.2021, 23:33, "Jay K" <<a href="mailto:jayk123@hotmail.com" rel="noopener noreferrer">jayk123@hotmail.com</a>>:<br /><br />     Try things like git checkout -p 8098c08<br />     or git checkout -p 8098c08~1<br />     -p to prompt for everything<br />     ~1 for prior commit.<br />     Given that you have a commit hash, the stuff is saved plenty.<br />     Unless you force push to github, that can erase stuff.<br />     Also git show 8098c08 and you can piece things together.<br />     git difftool -dy 8098c08~1..8098c08<br />     etc.<br />     Or checkout the branch where the revert happened.<br />     Revert the revert.<br />     Cherry pick the revert-revert to wherever.<br />     Many options.<br />       - Jay<br /> ------------------------------------------------------------------------<br />     *From:* M3devel <<a href="mailto:m3devel-bounces@elegosoft.com" rel="noopener noreferrer">m3devel-bounces@elegosoft.com</a><br />     <<a href="mailto:m3devel-bounces@elegosoft.com" rel="noopener noreferrer">mailto:m3devel-bounces@elegosoft.com</a>>> on behalf of Rodney Bates<br />     <<a href="mailto:rodney.bates@gmx.com" rel="noopener noreferrer">rodney.bates@gmx.com</a> <<a href="mailto:rodney.bates@gmx.com" rel="noopener noreferrer">mailto:rodney.bates@gmx.com</a>>><br />     *Sent:* Sunday, October 24, 2021 7:36 PM<br />     *To:* m3devel <<a href="mailto:m3devel@elegosoft.com" rel="noopener noreferrer">m3devel@elegosoft.com</a> <<a href="mailto:m3devel@elegosoft.com" rel="noopener noreferrer">mailto:m3devel@elegosoft.com</a>>><br />     *Subject:* [M3devel] Help unreverting<br />     According to my documentation on git, it protects me against loss of<br />     uncommited changes.  It has failed to do this, in my attempt to<br />     get 8098c08 and fe79f6c back, which were reverted by 556e65a, into<br />     new branch remoteConsructors, combined with latest fix ef1bd63.<br /><br />     While my local directory had unreverted versions of 8098c08 and fe79f6c,<br />     merged with what is now in ef1bd63 (in remoteConsructors only), the<br />     changes<br />     in the former two are now gone, neither in either branch nor in local<br />     copies.  There are many changes in these 3 commits, some of them in<br />     overlapping file sets.<br /><br />     I guess I was supposed to make a back up copy of my entire git repo<br />     before<br />     trying to create a branch and commit a new fix.<br /><br />     Does anybody have any advice on how to unravel this mess?<br />     _______________________________________________<br />     M3devel mailing list<br />     <a href="mailto:M3devel@elegosoft.com" rel="noopener noreferrer">M3devel@elegosoft.com</a> <<a href="mailto:M3devel@elegosoft.com" rel="noopener noreferrer">mailto:M3devel@elegosoft.com</a>><br /> <a href="https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fm3lists.elegosoft.com%2Fmailman%2Flistinfo%2Fm3devel&data=04%7C01%7C%7Cd495cc21639a47d0a6c608d99725aaac%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637707010307661153%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=W%2BFwWZPHUDMEGpKA%2BIFs0w6hqTuyYJ88uTOH1B7pbxs%3D&reserved=0" rel="noopener noreferrer">https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fm3lists.elegosoft.com%2Fmailman%2Flistinfo%2Fm3devel&amp;data=04%7C01%7C%7Cd495cc21639a47d0a6c608d99725aaac%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637707010307661153%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=W%2BFwWZPHUDMEGpKA%2BIFs0w6hqTuyYJ88uTOH1B7pbxs%3D&amp;reserved=0</a><br /> <<a href="https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fm3lists.elegosoft.com%2Fmailman%2Flistinfo%2Fm3devel&data=04%7C01%7C%7Cd495cc21639a47d0a6c608d99725aaac%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637707010307661153%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=W%2BFwWZPHUDMEGpKA%2BIFs0w6hqTuyYJ88uTOH1B7pbxs%3D&reserved=0" rel="noopener noreferrer">https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fm3lists.elegosoft.com%2Fmailman%2Flistinfo%2Fm3devel&data=04%7C01%7C%7Cd495cc21639a47d0a6c608d99725aaac%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637707010307661153%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=W%2BFwWZPHUDMEGpKA%2BIFs0w6hqTuyYJ88uTOH1B7pbxs%3D&reserved=0</a>><br />     ,<br /><br />     _______________________________________________<br />     M3devel mailing list<br />     <a href="mailto:M3devel@elegosoft.com" rel="noopener noreferrer">M3devel@elegosoft.com</a> <<a href="mailto:M3devel@elegosoft.com" rel="noopener noreferrer">mailto:M3devel@elegosoft.com</a>><br />     <a href="https://m3lists.elegosoft.com/mailman/listinfo/m3devel" rel="noopener noreferrer">https://m3lists.elegosoft.com/mailman/listinfo/m3devel</a><br /> <<a href="https://m3lists.elegosoft.com/mailman/listinfo/m3devel" rel="noopener noreferrer">https://m3lists.elegosoft.com/mailman/listinfo/m3devel</a>><br /> </blockquote></blockquote></blockquote>