Re: [Python-Dev] [Python-checkins] cpython (merge 3.2 -> default): Correctly merging #9319 into 3.3?
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 If a patch in 3.2 is not applicable in 3.3, a "null merge" should be done. If not, next developer tring to merge will find some other unrelated code to merge, and she doesn't have the context knowledge to know what to do :-). In this case, I merged code that doesn't actually compile, breaking the build for 20 minutes :-). And yes, I fully realized that I should try to compile locally first. Dealing with this unexpected merge when merging my own patch was... unexpected, and the code seemed sensible enough. Do we have some hat-of-shame I should wear because breaking the build? :). - -- Jesus Cea Avion _/_/ _/_/_/ _/_/_/ jcea@jcea.es - http://www.jcea.es/ _/_/ _/_/ _/_/ _/_/ _/_/ jabber / xmpp:jcea@jabber.org _/_/ _/_/ _/_/_/_/_/ . _/_/ _/_/ _/_/ _/_/ _/_/ "Things are not so easy" _/_/ _/_/ _/_/ _/_/ _/_/ _/_/ "My name is Dump, Core Dump" _/_/_/ _/_/_/ _/_/ _/_/ "El amor es poner tu felicidad en la felicidad de otro" - Leibniz -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iQCUAwUBTbTgpplgi5GaxT1NAQJqegP3QSVIf6yszZrFJEgKTaK4XXvHB965PdYN T9g8bx5IKXmiMjDBCatjuA2AAtwnL0Wd2Dw0tnGhRTqYHD2l+cMcFw/2JtV4L6sC c0fKm2o+V8gSW7KZwdvgNWiQlzE3lp2DiD/ng3gM3JlK/EKghIH8acDiJsHHrQtS 7T7iSLllOw== =+50u -----END PGP SIGNATURE-----
On Mon, 25 Apr 2011 04:47:03 +0200
Jesus Cea
And yes, I fully realized that I should try to compile locally first. Dealing with this unexpected merge when merging my own patch was... unexpected, and the code seemed sensible enough.
You should *always* recompile and run the affected tests before checking in a change. Even if the changes look "trivial". By trying to save a little time on your side your may lose a lot of other people's time.
Do we have some hat-of-shame I should wear because breaking the build? :).
The tests are still broken it seems: ====================================================================== ERROR: test_issue9319 (test.test_imp.ImportTests) ---------------------------------------------------------------------- Traceback (most recent call last): File "/Users/pythonbuildbot/buildarea/3.x.hansen-osx-x86-2/build/Lib/test/test_imp.py", line 181, in test_issue9319 imp.find_module, "test/badsyntax_pep3120") File "/Users/pythonbuildbot/buildarea/3.x.hansen-osx-x86-2/build/Lib/unittest/case.py", line 574, in assertRaises callableObj(*args, **kwargs) ImportError: No module named 'test/badsyntax_pep3120' Regards Antoine.
Le lundi 25 avril 2011 à 04:47 +0200, Jesus Cea a écrit :
If a patch in 3.2 is not applicable in 3.3, a "null merge" should be done.
Correct. Sorry, I forgot that. And yes, the 3.2 fix was not applicable to 3.3, that's why I forgot to merge.
If not, next developer tring to merge will find some other unrelated code to merge, and she doesn't have the context knowledge to know what to do :-)
Hum, you may read the history of the issue to decide what to do, or ask the commiter to do the merge.
In this case, I merged code that doesn't actually compile, breaking the build for 20 minutes :-).
He he, it was a trap! When you touch one of my commit, all buildbots turn red! :-)
Do we have some hat-of-shame I should wear because breaking the build? :).
Don't worry, it doesn't matter if you quickly fix your mistake. Victor
Hi,
If not, next developer tring to merge will find some other unrelated code to merge, and she doesn't have the context knowledge to know what to do :-).
Here’s a useful tip: instead of merging pulled changesets with your branch, do the reverse. That is: $ hg pull $ hg heads . # get only heads for the checked-out branch $ hg up other-head $ hg merge Now instead of merging unknown code into your checkout, you will merge the code added by your unpushed changesets to the other code. If you’re using a three-way file merge tool, it is your code that will be in the “other” pane, not the unknown code. Regards
participants (4)
-
Antoine Pitrou -
Jesus Cea -
Victor Stinner -
Éric Araujo