{"thread":{"id":"29913","subject":"stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","startedAt":"2012-03-12T06:48:55Z","lastAt":"2012-03-13T18:38:27Z","messageCount":37,"participants":["Jonathan Nieder","Willy Tarreau","Greg KH","Junio C Hamano","Thomas Rast","Sergio"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"186682","messageId":"20120312064855.GB16820@burratino","threadId":"29913","inReplyTo":"20120312063027.GB8971@1wt.eu","subject":"stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2012-03-12T06:48:55Z","receivedAt":"2012-03-12T06:48:55Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Hi,\n\n(adding git list and Thomas to cc)\nWilly Tarreau wrote:\n> On Sun, Mar 11, 2012 at 07:49:48PM -0700, Greg KH wrote:\n>> On Mon, Mar 12, 2012 at 01:07:26AM +0000, Ben Hutchings wrote:\n\n>>> The subject/first line should include '[IA64]', as in the original\n>>> commit. It looks like this has been automatically stripped.\n>>\n>> Yeah, munging patches to and from quilt and git will cause that to\n>> happen at times, it's quite common :(\n>\n> Indeed, and I've even changed my patch formats in haproxy to avoid brackets\n> due to this issue. The cause is that many patches are sent with a [PATCH]\n> prefix and that with Git, either you keep the subject line intact or you\n> remove everything that is between brackets. There's the -b option to only\n> remove remove tags looking like [PATCH], but my general experience with it\n> was not satisfying (I don't remind why).\n\nMaybe the problem was as simple as \"git am\" not knowing about \"-b\".\n\nTwo relevant patches:\n\n f7e5ea17 (am: learn passing -b to mailinfo, 2012-01-16)\n ee2d1cb4 (mailinfo: with -b, keep space after [foo], 2012-01-16)\n\nare in \"master\" and 1.7.10-rc0 and were not part of any earlier release.\n\nKudos to Thomas for writing them.\n\nJonathan\n"},{"id":"186687","messageId":"20120312085820.GA11569@1wt.eu","threadId":"29913","inReplyTo":"20120312064855.GB16820@burratino","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Willy Tarreau","fromEmail":"w@1wt.eu","sentAt":"2012-03-12T08:58:20Z","receivedAt":"2012-03-12T08:58:20Z","isPatch":true,"sender":{"key":"w@1wt.eu","avatar":"https://avatars.githubusercontent.com/u/8141789?v=4"},"body":"Hi Jonathan,\n\nOn Mon, Mar 12, 2012 at 01:48:55AM -0500, Jonathan Nieder wrote:\n> > Indeed, and I've even changed my patch formats in haproxy to avoid brackets\n> > due to this issue. The cause is that many patches are sent with a [PATCH]\n> > prefix and that with Git, either you keep the subject line intact or you\n> > remove everything that is between brackets. There's the -b option to only\n> > remove remove tags looking like [PATCH], but my general experience with it\n> > was not satisfying (I don't remind why).\n> \n> Maybe the problem was as simple as \"git am\" not knowing about \"-b\".\n\nI think you're quite right.\n\n> Two relevant patches:\n> \n> f7e5ea17 (am: learn passing -b to mailinfo, 2012-01-16)\n> ee2d1cb4 (mailinfo: with -b, keep space after [foo], 2012-01-16)\n> \n> are in \"master\" and 1.7.10-rc0 and were not part of any earlier release.\n>\n> Kudos to Thomas for writing them.\n\nAh, thank you very much for this useful info, I'll update my version !\n\nCheers,\nWilly\n"},{"id":"186724","messageId":"20120312152004.GB9380@kroah.com","threadId":"29913","inReplyTo":"20120312085820.GA11569@1wt.eu","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Greg KH","fromEmail":"greg@kroah.com","sentAt":"2012-03-12T15:20:04Z","receivedAt":"2012-03-12T15:20:04Z","isPatch":true,"sender":{"key":"greg@kroah.com","avatar":"https://gravatar.com/avatar/5bb5aa0cc2e01c00ec899d11130c07796bc186e465bae57bc34873b13b72c7c8?d=mp&s=160"},"body":"On Mon, Mar 12, 2012 at 09:58:20AM +0100, Willy Tarreau wrote:\n> Hi Jonathan,\n> \n> On Mon, Mar 12, 2012 at 01:48:55AM -0500, Jonathan Nieder wrote:\n> > > Indeed, and I've even changed my patch formats in haproxy to avoid brackets\n> > > due to this issue. The cause is that many patches are sent with a [PATCH]\n> > > prefix and that with Git, either you keep the subject line intact or you\n> > > remove everything that is between brackets. There's the -b option to only\n> > > remove remove tags looking like [PATCH], but my general experience with it\n> > > was not satisfying (I don't remind why).\n> > \n> > Maybe the problem was as simple as \"git am\" not knowing about \"-b\".\n> \n> I think you're quite right.\n> \n> > Two relevant patches:\n> > \n> > f7e5ea17 (am: learn passing -b to mailinfo, 2012-01-16)\n> > ee2d1cb4 (mailinfo: with -b, keep space after [foo], 2012-01-16)\n> > \n> > are in \"master\" and 1.7.10-rc0 and were not part of any earlier release.\n> >\n> > Kudos to Thomas for writing them.\n> \n> Ah, thank you very much for this useful info, I'll update my version !\n\nI don't see a -b option to 'git am' in the manpage, am I missing\nsomething here?\n\ngreg k-h\n"},{"id":"186725","messageId":"20120312152453.GB12405@1wt.eu","threadId":"29913","inReplyTo":"20120312152004.GB9380@kroah.com","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Willy Tarreau","fromEmail":"w@1wt.eu","sentAt":"2012-03-12T15:24:53Z","receivedAt":"2012-03-12T15:24:53Z","isPatch":true,"sender":{"key":"w@1wt.eu","avatar":"https://avatars.githubusercontent.com/u/8141789?v=4"},"body":"On Mon, Mar 12, 2012 at 08:20:04AM -0700, Greg KH wrote:\n> > > Two relevant patches:\n> > > \n> > > f7e5ea17 (am: learn passing -b to mailinfo, 2012-01-16)\n> > > ee2d1cb4 (mailinfo: with -b, keep space after [foo], 2012-01-16)\n> > > \n> > > are in \"master\" and 1.7.10-rc0 and were not part of any earlier release.\n> > >\n> > > Kudos to Thomas for writing them.\n> > \n> > Ah, thank you very much for this useful info, I'll update my version !\n> \n> I don't see a -b option to 'git am' in the manpage, am I missing\n> something here?\n\nIt's in the master tree only right now, and the option is \"--keep-non-patch\"\n(could have been shorter). Currently rebuilding to test it :-)\n\nWilly\n"},{"id":"186741","messageId":"7vfwddu60f.fsf@alter.siamese.dyndns.org","threadId":"29913","inReplyTo":"20120312152004.GB9380@kroah.com","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-12T16:40:48Z","receivedAt":"2012-03-12T16:40:48Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Greg KH writes:\n\n> I don't see a -b option to 'git am' in the manpage, am I missing\n> something here?\n\nAs this is a recent enhancement, it is very much appreciated if you\ntry out 1.7.10-rc0 (or 'master').\n"},{"id":"186742","messageId":"87aa3l4vqq.fsf@thomas.inf.ethz.ch","threadId":"29913","inReplyTo":"20120312152453.GB12405@1wt.eu","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Thomas Rast","fromEmail":"trast@inf.ethz.ch","sentAt":"2012-03-12T16:41:49Z","receivedAt":"2012-03-12T16:41:49Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"[+cc Junio because of backwards-compat issues]\n\nWilly Tarreau writes:\n\n> On Mon, Mar 12, 2012 at 08:20:04AM -0700, Greg KH wrote:\n>> \n>> I don't see a -b option to 'git am' in the manpage, am I missing\n>> something here?\n>\n> It's in the master tree only right now, and the option is \"--keep-non-patch\"\n> (could have been shorter). Currently rebuilding to test it :-)\n\nExactly.\n\nThe problem with -b is that it's a backwards-compatibility shorthand for\n--binary, which used to pass --allow-binary-replacement (or --binary) to\ngit-apply. However, that option was obsoleted in 2b6eef9 (Make apply\n--binary a no-op., 2006-09-06) and has been a no-op for over 5 years.\nIt has also not been documented since cb3a160 (git-am: ignore --binary\noption, 2008-08-09).\n\nSo perhaps we can safely claim -b for --keep-non-patch, like so:\n\ndiff --git i/Documentation/git-am.txt w/Documentation/git-am.txt\nindex ee6cca2..9ec9313 100644\n--- i/Documentation/git-am.txt\n+++ w/Documentation/git-am.txt\n@@ -40,6 +40,7 @@ OPTIONS\n --keep::\n \tPass `-k` flag to 'git mailinfo' (see linkgit:git-mailinfo[1]).\n \n+-b::\n --keep-non-patch::\n \tPass `-b` flag to 'git mailinfo' (see linkgit:git-mailinfo[1]).\n \ndiff --git i/git-am.sh w/git-am.sh\nindex 0bd290b..4b071de 100755\n--- i/git-am.sh\n+++ w/git-am.sh\n@@ -9,13 +9,13 @@ git am [options] [(|)...]\n git am [options] (--resolved | --skip | --abort)\n --\n i,interactive run interactively\n-b,binary* (historical option -- no-op)\n+binary* (historical option -- no-op)\n 3,3way allow fall back on 3way merging if needed\n q,quiet be quiet\n s,signoff add a Signed-off-by line to the commit message\n u,utf8 recode into utf8 (default)\n k,keep pass -k flag to git-mailinfo\n-keep-non-patch pass -b flag to git-mailinfo\n+b,keep-non-patch pass -b flag to git-mailinfo\n keep-cr pass --keep-cr flag to git-mailsplit for mbox format\n no-keep-cr do not pass --keep-cr flag to git-mailsplit independent of am.keepcr\n c,scissors strip everything before a scissors line\n@@ -379,7 +379,7 @@ do\n \tcase \"$1\" in\n \t-i|--interactive)\n \t\tinteractive=t ;;\n-\t-b|--binary)\n+\t--binary)\n \t\t: ;;\n \t-3|--3way)\n \t\tthreeway=t ;;\n@@ -391,7 +391,7 @@ do\n \t\tutf8= ;;\n \t-k|--keep)\n \t\tkeep=t ;;\n-\t--keep-non-patch)\n+\t-b|--keep-non-patch)\n \t\tkeep=b ;;\n \t-c|--scissors)\n \t\tscissors=t ;;\n\n-- \nThomas Rast\ntrast@{inf,student}.ethz.ch\n"},{"id":"186744","messageId":"20120312164812.GC12405@1wt.eu","threadId":"29913","inReplyTo":"7vfwddu60f.fsf@alter.siamese.dyndns.org","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Willy Tarreau","fromEmail":"w@1wt.eu","sentAt":"2012-03-12T16:48:12Z","receivedAt":"2012-03-12T16:48:12Z","isPatch":true,"sender":{"key":"w@1wt.eu","avatar":"https://avatars.githubusercontent.com/u/8141789?v=4"},"body":"Hi Junio,\n\nOn Mon, Mar 12, 2012 at 09:40:48AM -0700, Junio C Hamano wrote:\n> Greg KH writes:\n> \n> > I don't see a -b option to 'git am' in the manpage, am I missing\n> > something here?\n> \n> As this is a recent enhancement, it is very much appreciated if you\n> try out 1.7.10-rc0 (or 'master').\n\nI've just backported it to 1.7.9.3 (I'm not keen on living on the bleeding\nedge with my everyday tools), and it works nicely as expected.\n\nThanks!\nWilly\n"},{"id":"186745","messageId":"20120312165356.GD12405@1wt.eu","threadId":"29913","inReplyTo":"87aa3l4vqq.fsf@thomas.inf.ethz.ch","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Willy Tarreau","fromEmail":"w@1wt.eu","sentAt":"2012-03-12T16:53:56Z","receivedAt":"2012-03-12T16:53:56Z","isPatch":true,"sender":{"key":"w@1wt.eu","avatar":"https://avatars.githubusercontent.com/u/8141789?v=4"},"body":"On Mon, Mar 12, 2012 at 05:41:49PM +0100, Thomas Rast wrote:\n> [+cc Junio because of backwards-compat issues]\n> \n> Willy Tarreau writes:\n> \n> > On Mon, Mar 12, 2012 at 08:20:04AM -0700, Greg KH wrote:\n> >> \n> >> I don't see a -b option to 'git am' in the manpage, am I missing\n> >> something here?\n> >\n> > It's in the master tree only right now, and the option is \"--keep-non-patch\"\n> > (could have been shorter). Currently rebuilding to test it :-)\n> \n> Exactly.\n> \n> The problem with -b is that it's a backwards-compatibility shorthand for\n> --binary, which used to pass --allow-binary-replacement (or --binary) to\n> git-apply. However, that option was obsoleted in 2b6eef9 (Make apply\n> --binary a no-op., 2006-09-06) and has been a no-op for over 5 years.\n> It has also not been documented since cb3a160 (git-am: ignore --binary\n> option, 2008-08-09).\n> \n> So perhaps we can safely claim -b for --keep-non-patch, like so:\n\nYes I do think so, especially since 5 years ago, git commands were\ncalled hyphenated like \"git-am\" instead of \"git am\". So I don't think\nthere's any risk in reusing the option.\n\nRegards,\nWilly\n"},{"id":"186746","messageId":"20120312165703.GB18791@burratino","threadId":"29913","inReplyTo":"87aa3l4vqq.fsf@thomas.inf.ethz.ch","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2012-03-12T16:57:03Z","receivedAt":"2012-03-12T16:57:03Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Thomas Rast wrote:\n\n> The problem with -b is that it's a backwards-compatibility shorthand for\n> --binary, which used to pass --allow-binary-replacement (or --binary) to\n> git-apply. However, that option was obsoleted in 2b6eef9 (Make apply\n> --binary a no-op., 2006-09-06) and has been a no-op for over 5 years.\n> It has also not been documented since cb3a160 (git-am: ignore --binary\n> option, 2008-08-09).\n>\n> So perhaps we can safely claim -b for --keep-non-patch, like so:\n\nThanks.\n\nIt we want to be extra friendly to people who have been using\n\"format-patch --binary\" with \"am -b\" in their scripts, we could have a\ntransitional period during which -b is treated as a usage error.\n\nLuckily, a quick Google code search does not reveal any users for \"am\n-b\", so I am not too worried and would not mind your patch that just\nswitches over right away, though. After all, the failure modes are:\n\n - if my current script using \"am -b\" gets run using ancient git, it\n will accept binary patches and will strip out too many brackets\n in the subject line\n\n - if my ancient script using \"am -b\" gets run using current git, it\n will helpefully keep [IA64] brackets in the subject line\n\nNeither seems terribly painful.\n\nThe manual would need to mention that this once meant --binary to\navoid confusion when that happens.\n\nHope that helps,\nJonathan\n"},{"id":"186747","messageId":"20120312171257.GA25347@kroah.com","threadId":"29913","inReplyTo":"7vfwddu60f.fsf@alter.siamese.dyndns.org","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Greg KH","fromEmail":"greg@kroah.com","sentAt":"2012-03-12T17:12:57Z","receivedAt":"2012-03-12T17:12:57Z","isPatch":true,"sender":{"key":"greg@kroah.com","avatar":"https://gravatar.com/avatar/5bb5aa0cc2e01c00ec899d11130c07796bc186e465bae57bc34873b13b72c7c8?d=mp&s=160"},"body":"On Mon, Mar 12, 2012 at 09:40:48AM -0700, Junio C Hamano wrote:\n> Greg KH writes:\n> \n> > I don't see a -b option to 'git am' in the manpage, am I missing\n> > something here?\n> \n> As this is a recent enhancement, it is very much appreciated if you\n> try out 1.7.10-rc0 (or 'master').\n\nNice, I'll go do that now.\n\nDoes the flag propagate from 'git quiltimport'? That's how I import\npatches that would need this option most of the time.\n\nthanks,\n\ngreg k-h\n"},{"id":"186750","messageId":"7v4nttu2gf.fsf@alter.siamese.dyndns.org","threadId":"29913","inReplyTo":"20120312164812.GC12405@1wt.eu","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-12T17:57:36Z","receivedAt":"2012-03-12T17:57:36Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Willy Tarreau writes:\n\n> Hi Junio,\n>\n> On Mon, Mar 12, 2012 at 09:40:48AM -0700, Junio C Hamano wrote:\n>> Greg KH writes:\n>> \n>> > I don't see a -b option to 'git am' in the manpage, am I missing\n>> > something here?\n>> \n>> As this is a recent enhancement, it is very much appreciated if you\n>> try out 1.7.10-rc0 (or 'master').\n>\n> I've just backported it to 1.7.9.3 (I'm not keen on living on the bleeding\n> edge with my everyday tools), and it works nicely as expected.\n\nThis topic will be backported in later versions of 1.7.9.x track,\nbut living on the maintenance track does not have much smaller\nchance of breakage than living on the tip of 'master' these days,\nunless you are using distro packaged version. The usual rule of\nthumb if you are compiling from the source is that the tip of\n'master' is more stable than any tagged version, including the\nmaintenance track. See\n\nhttp://thread.gmane.org/gmane.comp.version-control.git/189657/focus=190814\n\nfor details.\n"},{"id":"186751","messageId":"7vzkblsnos.fsf@alter.siamese.dyndns.org","threadId":"29913","inReplyTo":"20120312171257.GA25347@kroah.com","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-12T18:01:55Z","receivedAt":"2012-03-12T18:01:55Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Greg KH writes:\n\n> Does the flag propagate from 'git quiltimport'?\n\nI didn't even know the script was shipped as part of the main\nPorcelain until I looked. I do not think anybody is maintaining it;\nthe last update was from the 1.6.2 era.\n\nA tested patch from quilt stakeholders is very much welcomed.\n\nThanks for bringing the script to my attention.\n"},{"id":"186752","messageId":"7vvcm9snko.fsf@alter.siamese.dyndns.org","threadId":"29913","inReplyTo":"20120312165703.GB18791@burratino","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-12T18:04:23Z","receivedAt":"2012-03-12T18:04:23Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jonathan Nieder writes:\n\n> Thomas Rast wrote:\n>\n>> The problem with -b is that it's a backwards-compatibility shorthand for\n>> --binary, which used to pass --allow-binary-replacement (or --binary) to\n>> git-apply. However, that option was obsoleted in 2b6eef9 (Make apply\n>> --binary a no-op., 2006-09-06) and has been a no-op for over 5 years.\n>> It has also not been documented since cb3a160 (git-am: ignore --binary\n>> option, 2008-08-09).\n>>\n>> So perhaps we can safely claim -b for --keep-non-patch, like so:\n\nWe can delete \"git am -b\" (as it was deprecated long time ago), wait\nfor a cycle or two, and then repurpose it. I do not mind starting\nthe first step (delete, but do not say anything about repurposing)\nbefore 1.7.10-rc1 happens.\n"},{"id":"186756","messageId":"20120312184549.GG12405@1wt.eu","threadId":"29913","inReplyTo":"7v4nttu2gf.fsf@alter.siamese.dyndns.org","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Willy Tarreau","fromEmail":"w@1wt.eu","sentAt":"2012-03-12T18:45:49Z","receivedAt":"2012-03-12T18:45:49Z","isPatch":true,"sender":{"key":"w@1wt.eu","avatar":"https://avatars.githubusercontent.com/u/8141789?v=4"},"body":"On Mon, Mar 12, 2012 at 10:57:36AM -0700, Junio C Hamano wrote:\n> > I've just backported it to 1.7.9.3 (I'm not keen on living on the bleeding\n> > edge with my everyday tools), and it works nicely as expected.\n> \n> This topic will be backported in later versions of 1.7.9.x track,\n> but living on the maintenance track does not have much smaller\n> chance of breakage than living on the tip of 'master' these days,\n> unless you are using distro packaged version. The usual rule of\n> thumb if you are compiling from the source is that the tip of\n> 'master' is more stable than any tagged version, including the\n> maintenance track. See\n> \n> http://thread.gmane.org/gmane.comp.version-control.git/189657/focus=190814\n> \n> for details.\n\nI know, but you see I was running on 1.7.2.3. Generally if one version\nworks for me, I don't upgrade it for a year or two. I've been hit a few\ntimes in the past by some quite annoying bugs (as is to be expected from\nany software in the development branch), and lost a lot of time on this.\nRest assured that I don't feel comfortable on 1.7.9 either, that's a big\njump for me but I know that most often it works quite well :-)\n\nWilly\n"},{"id":"186757","messageId":"20120312185008.GH12405@1wt.eu","threadId":"29913","inReplyTo":"7vvcm9snko.fsf@alter.siamese.dyndns.org","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Willy Tarreau","fromEmail":"w@1wt.eu","sentAt":"2012-03-12T18:50:08Z","receivedAt":"2012-03-12T18:50:08Z","isPatch":true,"sender":{"key":"w@1wt.eu","avatar":"https://avatars.githubusercontent.com/u/8141789?v=4"},"body":"On Mon, Mar 12, 2012 at 11:04:23AM -0700, Junio C Hamano wrote:\n> >> So perhaps we can safely claim -b for --keep-non-patch, like so:\n> \n> We can delete \"git am -b\" (as it was deprecated long time ago), wait\n> for a cycle or two, and then repurpose it. I do not mind starting\n> the first step (delete, but do not say anything about repurposing)\n> before 1.7.10-rc1 happens.\n\n>From my user experience and what I see on a number of coworkers, users\ntend to make big jumps when they need a new feature, so in practice,\nnot offering the option in a version or two would probably not affect\nmost users, especially the ones still relying on the old behaviour. So\nI don't see much benefit in waiting for repurposing the option.\n\nJust my 2 cents,\nWilly\n"},{"id":"186760","messageId":"20120312185440.GA21817@burratino","threadId":"29913","inReplyTo":"20120312185008.GH12405@1wt.eu","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2012-03-12T18:54:40Z","receivedAt":"2012-03-12T18:54:40Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Willy Tarreau wrote:\n\n> From my user experience and what I see on a number of coworkers, users\n> tend to make big jumps when they need a new feature, so in practice,\n> not offering the option in a version or two would probably not affect\n> most users, especially the ones still relying on the old behaviour. So\n> I don't see much benefit in waiting for repurposing the option.\n\nThe benefit is that if it does turn out to be a regression, early\nadopters will complain to us because their scripts have stopped\nworking and we get a chance to back out the change without having to\nworry about others who have started to rely on the option.\n\nRegards,\nJonathan\n"},{"id":"186764","messageId":"20120312191727.GI12405@1wt.eu","threadId":"29913","inReplyTo":"20120312185440.GA21817@burratino","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Willy Tarreau","fromEmail":"w@1wt.eu","sentAt":"2012-03-12T19:17:27Z","receivedAt":"2012-03-12T19:17:27Z","isPatch":true,"sender":{"key":"w@1wt.eu","avatar":"https://avatars.githubusercontent.com/u/8141789?v=4"},"body":"On Mon, Mar 12, 2012 at 01:54:40PM -0500, Jonathan Nieder wrote:\n> Willy Tarreau wrote:\n> \n> > From my user experience and what I see on a number of coworkers, users\n> > tend to make big jumps when they need a new feature, so in practice,\n> > not offering the option in a version or two would probably not affect\n> > most users, especially the ones still relying on the old behaviour. So\n> > I don't see much benefit in waiting for repurposing the option.\n> \n> The benefit is that if it does turn out to be a regression, early\n> adopters will complain to us because their scripts have stopped\n> working and we get a chance to back out the change without having to\n> worry about others who have started to rely on the option.\n\nIndeed you're right. I forgot the case of the old script relying\non the silently ignored parameter!\n\nRegards,\nWilly\n"},{"id":"186765","messageId":"20120312192627.GA24873@kroah.com","threadId":"29913","inReplyTo":"7vzkblsnos.fsf@alter.siamese.dyndns.org","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Greg KH","fromEmail":"greg@kroah.com","sentAt":"2012-03-12T19:26:27Z","receivedAt":"2012-03-12T19:26:27Z","isPatch":true,"sender":{"key":"greg@kroah.com","avatar":"https://gravatar.com/avatar/5bb5aa0cc2e01c00ec899d11130c07796bc186e465bae57bc34873b13b72c7c8?d=mp&s=160"},"body":"On Mon, Mar 12, 2012 at 11:01:55AM -0700, Junio C Hamano wrote:\n> Greg KH writes:\n> \n> > Does the flag propagate from 'git quiltimport'?\n> \n> I didn't even know the script was shipped as part of the main\n> Porcelain until I looked. I do not think anybody is maintaining it;\n> the last update was from the 1.6.2 era.\n\nAh, so my feeling that I'm the only one using it is true :)\n\n> A tested patch from quilt stakeholders is very much welcomed.\n\nI'll be glad to maintain this if you need me to. I'll poke around with\nit tomorrow to see what needs to be done here.\n\nthanks,\n\ngreg k-h\n"},{"id":"186766","messageId":"7vboo1sjnl.fsf@alter.siamese.dyndns.org","threadId":"29913","inReplyTo":"20120312184549.GG12405@1wt.eu","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-12T19:29:02Z","receivedAt":"2012-03-12T19:29:02Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Willy Tarreau writes:\n\n> Rest assured that I don't feel comfortable on 1.7.9 either, that's a big\n> jump for me but I know that most often it works quite well :-)\n\nIt being a big jump is not a problem I can solve, as it is up to you\nto deliberately stay behind until you decide to make a big jump.\n\nI can indirectly solve it by not backporting as many fixes to the\nmaintenance tracks, but I am too nice to be playing that nasty ;-).\n"},{"id":"186770","messageId":"7vty1tr418.fsf@alter.siamese.dyndns.org","threadId":"29913","inReplyTo":"20120312192627.GA24873@kroah.com","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-12T19:51:47Z","receivedAt":"2012-03-12T19:51:47Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Greg KH writes:\n\n> On Mon, Mar 12, 2012 at 11:01:55AM -0700, Junio C Hamano wrote:\n>> Greg KH writes:\n>> \n>> > Does the flag propagate from 'git quiltimport'?\n>> \n>> I didn't even know the script was shipped as part of the main\n>> Porcelain until I looked. I do not think anybody is maintaining it;\n>> the last update was from the 1.6.2 era.\n>\n> Ah, so my feeling that I'm the only one using it is true :)\n>\n>> A tested patch from quilt stakeholders is very much welcomed.\n>\n> I'll be glad to maintain this if you need me to. I'll poke around with\n> it tomorrow to see what needs to be done here.\n\nSurely, and thanks.\n"},{"id":"186775","messageId":"20120312201915.GK12405@1wt.eu","threadId":"29913","inReplyTo":"20120312192627.GA24873@kroah.com","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Willy Tarreau","fromEmail":"w@1wt.eu","sentAt":"2012-03-12T20:19:15Z","receivedAt":"2012-03-12T20:19:15Z","isPatch":true,"sender":{"key":"w@1wt.eu","avatar":"https://avatars.githubusercontent.com/u/8141789?v=4"},"body":"On Mon, Mar 12, 2012 at 12:26:27PM -0700, Greg KH wrote:\n> On Mon, Mar 12, 2012 at 11:01:55AM -0700, Junio C Hamano wrote:\n> > Greg KH writes:\n> > \n> > > Does the flag propagate from 'git quiltimport'?\n> > \n> > I didn't even know the script was shipped as part of the main\n> > Porcelain until I looked. I do not think anybody is maintaining it;\n> > the last update was from the 1.6.2 era.\n> \n> Ah, so my feeling that I'm the only one using it is true :)\n\nWell, we're two precisely :-)\n\n> > A tested patch from quilt stakeholders is very much welcomed.\n> \n> I'll be glad to maintain this if you need me to. I'll poke around with\n> it tomorrow to see what needs to be done here.\n\nThat would be great, this tool is really handy.\n\nCheers,\nWilly\n"},{"id":"186790","messageId":"20120312214139.GA11362@burratino","threadId":"29913","inReplyTo":"m38vj5zeo5.fsf_-_@localhost.localdomain","subject":"Re: Removing unmaintained scripts","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2012-03-12T21:41:39Z","receivedAt":"2012-03-12T21:41:39Z","isPatch":false,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Jakub Narebski wrote:\n\n> A'propos, there is git-archimport script, last modified (as an\n> individual script) in 2007. Is anyone using GNU arch nowadays;\n> isn't arch-fast-export better solution anyway?\n\nAccording to [1] (see the green line), it still has some users,\nor at least people who access the file now and then. ;-) As long\nas the script works, why would we want to cause them trouble?\n\nFor comparison, here's the corresponding graph for people using\nGNU arch[2] and participating in the same survey.\n\nHope that helps,\nJonathan\n\n[1] http://qa.debian.org/popcon-graph.php?packages=git-arch\n http://qa.debian.org/popcon.php?package=git-core\n[2] http://qa.debian.org/popcon-graph.php?packages=tla\n http://qa.debian.org/popcon.php?package=tla\n"},{"id":"186791","messageId":"87399dpk48.fsf@thomas.inf.ethz.ch","threadId":"29913","inReplyTo":"7vvcm9snko.fsf@alter.siamese.dyndns.org","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Thomas Rast","fromEmail":"trast@inf.ethz.ch","sentAt":"2012-03-12T21:47:19Z","receivedAt":"2012-03-12T21:47:19Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"Junio C Hamano writes:\n\n> Jonathan Nieder writes:\n>\n>> Thomas Rast wrote:\n>>\n>>> The problem with -b is that it's a backwards-compatibility shorthand for\n>>> --binary, which used to pass --allow-binary-replacement (or --binary) to\n>>> git-apply. However, that option was obsoleted in 2b6eef9 (Make apply\n>>> --binary a no-op., 2006-09-06) and has been a no-op for over 5 years.\n>>> It has also not been documented since cb3a160 (git-am: ignore --binary\n>>> option, 2008-08-09).\n>>>\n>>> So perhaps we can safely claim -b for --keep-non-patch, like so:\n>\n> We can delete \"git am -b\" (as it was deprecated long time ago), wait\n> for a cycle or two, and then repurpose it. I do not mind starting\n> the first step (delete, but do not say anything about repurposing)\n> before 1.7.10-rc1 happens.\n\nOk, but if I don't get to say anything about repurposing, can I at least\nmake it an error message instead of just spewing out the \"invalid\noption\" help?\n\n----- 8< -----\nSubject: [PATCH] git-am: error out when seeing -b/--binary\n\nThe --binary option to git-apply has been a no-op since 2b6eef9 (Make\napply --binary a no-op., 2006-09-06) and was deprecated in cb3a160\n(git-am: ignore --binary option, 2008-08-09).\n\nWe could remove it outright, but let's be nice to people who still\nhave scripts saying 'git am -b' (if they exist) and tell them the\nreason for the sudden failure.\n\nSigned-off-by: Thomas Rast \n---\n git-am.sh | 4 +++-\n 1 file changed, 3 insertions(+), 1 deletion(-)\n\ndiff --git a/git-am.sh b/git-am.sh\nindex 0bd290b..faae820 100755\n--- a/git-am.sh\n+++ b/git-am.sh\n@@ -380,7 +380,9 @@ do\n \t-i|--interactive)\n \t\tinteractive=t ;;\n \t-b|--binary)\n-\t\t: ;;\n+\t\techo >&2 \"The -b/--binary option was deprecated in 1.6.0 and removed in 1.7.10.\"\n+\t\tdie \"Please adjust your scripts.\"\n+\t\t;;\n \t-3|--3way)\n \t\tthreeway=t ;;\n \t-s|--signoff)\n\n-- \nThomas Rast\ntrast@{inf,student}.ethz.ch\n"},{"id":"186794","messageId":"20120312215607.GB11362@burratino","threadId":"29913","inReplyTo":"87399dpk48.fsf@thomas.inf.ethz.ch","subject":"Re: [PATCH] git-am: error out when seeing -b/--binary","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2012-03-12T21:56:07Z","receivedAt":"2012-03-12T21:56:07Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Thomas Rast wrote:\n\n> Ok, but if I don't get to say anything about repurposing, can I at least\n> make it an error message instead of just spewing out the \"invalid\n> option\" help?\n\nI don't mind either way.\n\n[...]\n> --- a/git-am.sh\n> +++ b/git-am.sh\n> @@ -380,7 +380,9 @@ do\n> \t-i|--interactive)\n> \t\tinteractive=t ;;\n> \t-b|--binary)\n> -\t\t: ;;\n> +\t\techo >&2 \"The -b/--binary option was deprecated in 1.6.0 and removed in 1.7.10.\"\n> +\t\tdie \"Please adjust your scripts.\"\n> +\t\t;;\n\nHm, on second thought, if people are seeing this message, I would\nprefer if they write to the mailing list so we can find out about it.\nSo I really would rather see this say\n\n\t--binary)\n\t\t: ;;\n\nand have \"-b\" completely unrecognized, without any words in our\ndefense except for a note in the release notes mentioning the option's\nremoval and that it has been an unadvertised backward-compatibility\nno-op since 1.6.0.\n\nJonathan\n"},{"id":"186795","messageId":"7v399dpjnd.fsf@alter.siamese.dyndns.org","threadId":"29913","inReplyTo":"87399dpk48.fsf@thomas.inf.ethz.ch","subject":"Re: stripping [PATCH] without losing later tags from mailed patches (Re: [ 02/12] Remove COMPAT_IA32 support)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-12T21:57:26Z","receivedAt":"2012-03-12T21:57:26Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thomas Rast writes:\n\n>> We can delete \"git am -b\" (as it was deprecated long time ago), wait\n>> for a cycle or two, and then repurpose it. I do not mind starting\n>> the first step (delete, but do not say anything about repurposing)\n>> before 1.7.10-rc1 happens.\n>\n> Ok, but if I don't get to say anything about repurposing, can I at least\n> make it an error message instead of just spewing out the \"invalid\n> option\" help?\n\nSurely. It is not \"Am I at least allowed to say\" at all; we really\n_should_ make it clear why it is no longer supported.\n\nWhat I meant was that \"it will start meaning something different\" is\nnot a relevant thing to see when somebody sees his old script that\nuses \"-b\" breaks.\n\nLet's apply your patch so that we do not have to wait one cycle too\nlong.\n\nThanks.\n\n>\n> ----- 8< -----\n> Subject: [PATCH] git-am: error out when seeing -b/--binary\n>\n> The --binary option to git-apply has been a no-op since 2b6eef9 (Make\n> apply --binary a no-op., 2006-09-06) and was deprecated in cb3a160\n> (git-am: ignore --binary option, 2008-08-09).\n>\n> We could remove it outright, but let's be nice to people who still\n> have scripts saying 'git am -b' (if they exist) and tell them the\n> reason for the sudden failure.\n>\n> Signed-off-by: Thomas Rast \n> ---\n> git-am.sh | 4 +++-\n> 1 file changed, 3 insertions(+), 1 deletion(-)\n>\n> diff --git a/git-am.sh b/git-am.sh\n> index 0bd290b..faae820 100755\n> --- a/git-am.sh\n> +++ b/git-am.sh\n> @@ -380,7 +380,9 @@ do\n> \t-i|--interactive)\n> \t\tinteractive=t ;;\n> \t-b|--binary)\n> -\t\t: ;;\n> +\t\techo >&2 \"The -b/--binary option was deprecated in 1.6.0 and removed in 1.7.10.\"\n> +\t\tdie \"Please adjust your scripts.\"\n> +\t\t;;\n> \t-3|--3way)\n> \t\tthreeway=t ;;\n> \t-s|--signoff)\n"},{"id":"186796","messageId":"874ntto4t8.fsf@thomas.inf.ethz.ch","threadId":"29913","inReplyTo":"20120312215607.GB11362@burratino","subject":"Re: [PATCH] git-am: error out when seeing -b/--binary","fromName":"Thomas Rast","fromEmail":"trast@inf.ethz.ch","sentAt":"2012-03-12T22:03:15Z","receivedAt":"2012-03-12T22:03:15Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"Jonathan Nieder writes:\n\n> Hm, on second thought, if people are seeing this message, I would\n> prefer if they write to the mailing list so we can find out about it.\n> So I really would rather see this say\n>\n> \t--binary)\n> \t\t: ;;\n>\n> and have \"-b\" completely unrecognized, without any words in our\n> defense except for a note in the release notes mentioning the option's\n> removal and that it has been an unadvertised backward-compatibility\n> no-op since 1.6.0.\n\nI'd hate doing that, mostly because other projects got me really angry\nabout similar issues, e.g., 71c020c (Disable asciidoc 8.4.1+ semantics\nfor `{plus}` and friends, 2009-07-25).\n\nBy the time I knew what the problem was, I figured posting anywhere was\nuseless since the change was already in the wild, and thus needed\nworking around on our end; and all I could possibly post was an angry\nletter saying how unhappy I was about their work.\n\nI didn't, and worked around it. But that was after a lot of frustrated\ninvestigation. So I'd rather not do the same to our unlucky users.\n\n-- \nThomas Rast\ntrast@{inf,student}.ethz.ch\n"},{"id":"186797","messageId":"7vy5r5o4ea.fsf@alter.siamese.dyndns.org","threadId":"29913","inReplyTo":"20120312215607.GB11362@burratino","subject":"Re: [PATCH] git-am: error out when seeing -b/--binary","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-12T22:12:13Z","receivedAt":"2012-03-12T22:12:13Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jonathan Nieder writes:\n\n> Hm, on second thought, if people are seeing this message, I would\n> prefer if they write to the mailing list so we can find out about it.\n> So I really would rather see this say\n>\n> \t--binary)\n> \t\t: ;;\n>\n> and have \"-b\" completely unrecognized, without any words in our\n> defense except for a note in the release notes mentioning the option's\n> removal and that it has been an unadvertised backward-compatibility\n> no-op since 1.6.0.\n\nI do not mind keeping --binary working intact, but I think an\napproach to say that \"-b\" no longer works and is finally removed\nvery firmly is a very sane one. We would be getting a slightly\nbetter feel of how stale the people's script could be with your\napproach, but at the same time, we would be annoying far more people\nwho do not even know that there are some people passionately trying\nto make the users' Git life better, or where these people are\nhanging out.\n\nI do not think removal of a no-op \"-b\" is something subject to\nvoting at this point, so the only thing you would get from the\nbetter feel of the user universe is when we can repurpose the option\nsafely.\n"},{"id":"186798","messageId":"20120312222227.GC11362@burratino","threadId":"29913","inReplyTo":"874ntto4t8.fsf@thomas.inf.ethz.ch","subject":"Re: [PATCH] git-am: error out when seeing -b/--binary","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2012-03-12T22:22:27Z","receivedAt":"2012-03-12T22:22:27Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Thomas Rast wrote:\n> Jonathan Nieder writes:\n\n>> Hm, on second thought, if people are seeing this message, I would\n>> prefer if they write to the mailing list so we can find out about it.\n>> So I really would rather see this say\n>>\n>> \t--binary)\n>> \t\t: ;;\n>>\n>> and have \"-b\" completely unrecognized, without any words in our\n>> defense except for a note in the release notes mentioning the option's\n>> removal and that it has been an unadvertised backward-compatibility\n>> no-op since 1.6.0.\n>\n> I'd hate doing that, mostly because other projects got me really angry\n> about similar issues, e.g., 71c020c (Disable asciidoc 8.4.1+ semantics\n> for `{plus}` and friends, 2009-07-25).\n\nOh, now that I think about it that way, you're definitely right.\n\nSo, how about something like this?\n\n\t--binary)\n\t\t: ;;\n\t-b)\n\t\tgettextln >&2 \"The -b option (a no-op short for --binary) was removed in 1.7.10.\"\n\t\tdie \"$(gettext \"Please adjust your scripts.\")\"\n\t\t;;\n\nMentioning deprecation in 1.6.0 in the message left me uneasy because\nwe never actually did anything to actively deprecate the option; it\njust has not been needed since 1.4.3 and we stopped advertising it in\nthe manpage in 1.6.0. So I don't like the implication of \"this is all\nright because we told you so\" --- on the contrary, it is \"in practice\nnobody seems to be using this option and we hope nobody will notice\nwhen we take it away\".\n\nJonathan\n"},{"id":"186823","messageId":"loom.20120313T102017-876@post.gmane.org","threadId":"29913","inReplyTo":"20120312214139.GA11362@burratino","subject":"Re: Removing unmaintained scripts","fromName":"Sergio","fromEmail":"sergio.callegari@gmail.com","sentAt":"2012-03-13T09:23:34Z","receivedAt":"2012-03-13T09:23:34Z","isPatch":false,"sender":{"key":"sergio.callegari@gmail.com","avatar":"https://gravatar.com/avatar/c98f41317e0422c1e630385de0e3970227b8e5ad15f35ba8586066467cc833bc?d=mp&s=160"},"body":"Jonathan Nieder gmail.com> writes:\n\n> \n> Jakub Narebski wrote:\n> \n> > A'propos, there is git-archimport script, last modified (as an\n> > individual script) in 2007. Is anyone using GNU arch nowadays;\n> > isn't arch-fast-export better solution anyway?\n> \n> According to [1] (see the green line), it still has some users,\n> or at least people who access the file now and then. As long\n> as the script works, why would we want to cause them trouble?\n> \n> For comparison, here's the corresponding graph for people using\n> GNU arch[2] and participating in the same survey.\n> \n> Hope that helps,\n> Jonathan\n> \n> [1] http://qa.debian.org/popcon-graph.php?packages=git-arch\n> http://qa.debian.org/popcon.php?package=git-core\n> [2] http://qa.debian.org/popcon-graph.php?packages=tla\n> http://qa.debian.org/popcon.php?package=tla\n> \n\n\nHi,\n\nplease do not remove the ability to convert arch/tla repos unless it is a real\ncost to maintain the scripts. There may be few people who still use arch, but\nthere may be a lot of them who have arch repos of quiescent stuff that one day\nthey may need to revitalize.\n\nSergio\n"},{"id":"186825","messageId":"87k42olugb.fsf@thomas.inf.ethz.ch","threadId":"29913","inReplyTo":"loom.20120313T102017-876@post.gmane.org","subject":"Re: Removing unmaintained scripts","fromName":"Thomas Rast","fromEmail":"trast@student.ethz.ch","sentAt":"2012-03-13T09:29:56Z","receivedAt":"2012-03-13T09:29:56Z","isPatch":false,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"Sergio writes:\n\n> Jonathan Nieder gmail.com> writes:\n>\n>> > A'propos, there is git-archimport script, last modified (as an\n>> > individual script) in 2007. Is anyone using GNU arch nowadays;\n>> > isn't arch-fast-export better solution anyway?\n>\n> please do not remove the ability to convert arch/tla repos unless it is a real\n> cost to maintain the scripts. There may be few people who still use arch, but\n> there may be a lot of them who have arch repos of quiescent stuff that one day\n> they may need to revitalize.\n\nPerhaps it would be a good idea to add a test to ensure that\ngit-archimport doesn't suddenly break with nobody noticing for months.\nCan you contribute a reasonable (small but representative) test repo?\n\n-- \nThomas Rast\ntrast@{inf,student}.ethz.ch\n"},{"id":"186870","messageId":"87fwdcldqj.fsf@thomas.inf.ethz.ch","threadId":"29913","inReplyTo":"20120312222227.GC11362@burratino","subject":"Re: [PATCH] git-am: error out when seeing -b/--binary","fromName":"Thomas Rast","fromEmail":"trast@student.ethz.ch","sentAt":"2012-03-13T15:31:00Z","receivedAt":"2012-03-13T15:31:00Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"Jonathan Nieder writes:\n\n> \t--binary)\n> \t\t: ;;\n> \t-b)\n> \t\tgettextln >&2 \"The -b option (a no-op short for --binary) was removed in 1.7.10.\"\n> \t\tdie \"$(gettext \"Please adjust your scripts.\")\"\n> \t\t;;\n>\n> Mentioning deprecation in 1.6.0 in the message left me uneasy because\n> we never actually did anything to actively deprecate the option; it\n> just has not been needed since 1.4.3 and we stopped advertising it in\n> the manpage in 1.6.0. So I don't like the implication of \"this is all\n> right because we told you so\" --- on the contrary, it is \"in practice\n> nobody seems to be using this option and we hope nobody will notice\n> when we take it away\".\n\nHmm, I had an alternate patch ready in the morning, but Junio beat us to\nit and applied the old one to master.\n\nI don't really think it matters enough to apply *another*, so I'll leave\nit at that.\n\n-- \nThomas Rast\ntrast@{inf,student}.ethz.ch\n"},{"id":"186871","messageId":"20120313154209.GC1978@burratino","threadId":"29913","inReplyTo":"87fwdcldqj.fsf@thomas.inf.ethz.ch","subject":"Re: [PATCH] git-am: error out when seeing -b/--binary","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2012-03-13T15:42:09Z","receivedAt":"2012-03-13T15:42:09Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"(resetting cc list)\nThomas Rast wrote:\n\n> Hmm, I had an alternate patch ready in the morning, but Junio beat us to\n> it and applied the old one to master.\n>\n> I don't really think it matters enough to apply *another*, so I'll leave\n> it at that.\n\nMay I have a copy? If it makes the wording better somehow, I would be\nhappy to do the work of making a patch on top of master using it.\n"},{"id":"186872","messageId":"8762e8ld5b.fsf@thomas.inf.ethz.ch","threadId":"29913","inReplyTo":"20120313154209.GC1978@burratino","subject":"Re: [PATCH] git-am: error out when seeing -b/--binary","fromName":"Thomas Rast","fromEmail":"trast@inf.ethz.ch","sentAt":"2012-03-13T15:43:44Z","receivedAt":"2012-03-13T15:43:44Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"Jonathan Nieder writes:\n\n> (resetting cc list)\n> Thomas Rast wrote:\n>\n>> Hmm, I had an alternate patch ready in the morning, but Junio beat us to\n>> it and applied the old one to master.\n>>\n>> I don't really think it matters enough to apply *another*, so I'll leave\n>> it at that.\n>\n> May I have a copy? If it makes the wording better somehow, I would be\n> happy to do the work of making a patch on top of master using it.\n\nSure, but it's nothing magic :-)\n\n------ 8< ------\nSubject: [PATCH] git-am: error out when seeing -b/--binary\n\nThe --binary option to git-apply has been a no-op since 2b6eef9 (Make\napply --binary a no-op., 2006-09-06) and was deprecated in cb3a160\n(git-am: ignore --binary option, 2008-08-09).\n\nWe could remove it outright, but let's be nice to people who still\nhave scripts saying 'git am -b' (if they exist) and tell them the\nreason for the sudden failure.\n\nSigned-off-by: Thomas Rast \n---\n git-am.sh | 6 +++++-\n 1 file changed, 5 insertions(+), 1 deletion(-)\n\ndiff --git a/git-am.sh b/git-am.sh\nindex 0bd290b..e1bdbfc 100755\n--- a/git-am.sh\n+++ b/git-am.sh\n@@ -379,8 +379,12 @@ do\n \tcase \"$1\" in\n \t-i|--interactive)\n \t\tinteractive=t ;;\n-\t-b|--binary)\n+\t--binary)\n \t\t: ;;\n+\t-b)\n+\t\tgettextln >&2 \"The -b option (shorthand for --binary, both no-ops) was removed in 1.7.10.\"\n+\t\tdie \"$(gettext \"Please adjust your scripts.\")\"\n+\t\t;;\n \t-3|--3way)\n \t\tthreeway=t ;;\n \t-s|--signoff)\n\n\n-- \nThomas Rast\ntrast@{inf,student}.ethz.ch\n"},{"id":"186874","messageId":"7v62e8la9y.fsf@alter.siamese.dyndns.org","threadId":"29913","inReplyTo":"87fwdcldqj.fsf@thomas.inf.ethz.ch","subject":"Re: [PATCH] git-am: error out when seeing -b/--binary","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-13T17:31:03Z","receivedAt":"2012-03-13T17:31:03Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thomas Rast writes:\n\n> Jonathan Nieder writes:\n>\n>> \t--binary)\n>> \t\t: ;;\n>> \t-b)\n>> \t\tgettextln >&2 \"The -b option (a no-op short for --binary) was removed in 1.7.10.\"\n>> \t\tdie \"$(gettext \"Please adjust your scripts.\")\"\n>> \t\t;;\n>>\n>> Mentioning deprecation in 1.6.0 in the message left me uneasy because\n>> we never actually did anything to actively deprecate the option; it\n>> just has not been needed since 1.4.3 and we stopped advertising it in\n>> the manpage in 1.6.0. So I don't like the implication of \"this is all\n>> right because we told you so\" --- on the contrary, it is \"in practice\n>> nobody seems to be using this option and we hope nobody will notice\n>> when we take it away\".\n>\n> Hmm, I had an alternate patch ready in the morning, but Junio beat us to\n> it and applied the old one to master.\n\nI really don't think it is a good idea to avoid mentioning 1.6.0, at\nwhich we *removed* description of the option in our manual pages and\nfrom the \"git am -h\" help message. How much more active deprecation\nwould a user want?\n\nTo put it another way, think what your answer would be when somebody\nsees the message and says \"eh? all of a sudden it was removed?\".\nWouldn't you tell him \"At 1.6.0 we deprecated it and stopped\nadvertising it\"? Why not give that answer upfront?\n\nEspecially when you think \"in practice nobody seems to be using\nthis\" is true?\n"},{"id":"186882","messageId":"20120313175158.GA6653@burratino","threadId":"29913","inReplyTo":"7v62e8la9y.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] git-am: error out when seeing -b/--binary","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2012-03-13T17:51:58Z","receivedAt":"2012-03-13T17:51:58Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Junio C Hamano wrote:\n\n> I really don't think it is a good idea to avoid mentioning 1.6.0, at\n> which we *removed* description of the option in our manual pages and\n> from the \"git am -h\" help message. How much more active deprecation\n> would a user want?\n\nA warning when the option is used or a mention in the release notes.\n\n> To put it another way, think what your answer would be when somebody\n> sees the message and says \"eh? all of a sudden it was removed?\".\n> Wouldn't you tell him \"At 1.6.0 we deprecated it and stopped\n> advertising it\"? Why not give that answer upfront?\n\nI would tell her \"Since 1.4.3 it has been a compatibility no-op and\nour documentation made that clear, and by now based on a search nobody\nseems to be using it\".\n\nWhat happened in 1.6.0, then? Well, before 1.6.0, the git-am(1)\nmanual said\n\n -b, --binary\n\tPass --allow-binary-replacement flag to git-apply (see\n\tgit-apply(1)).\n\nand the git-apply(1) manual said\n\n --allow-binary-replacement, --binary\n\tHistorically we did not allow binary patch applied\n\twithout an explicit permission from the user, and this\n\tflag was the way to do so. Currently we always allow binary\n\tpatch application, so this is a no-op.\n\nAfterwards, the entry in the git-am(1) manual was removed, saving the\nreader a little time.\n\nJonathan\n"},{"id":"186887","messageId":"7vipi8wect.fsf@alter.siamese.dyndns.org","threadId":"29913","inReplyTo":"20120313175158.GA6653@burratino","subject":"Re: [PATCH] git-am: error out when seeing -b/--binary","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-13T18:22:10Z","receivedAt":"2012-03-13T18:22:10Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jonathan Nieder writes:\n\n>> I really don't think it is a good idea to avoid mentioning 1.6.0, at\n>> which we *removed* description of the option in our manual pages and\n>> from the \"git am -h\" help message. How much more active deprecation\n>> would a user want?\n>\n> A warning when the option is used or a mention in the release notes.\n\nOk, then we should probably instead do these two starting at 1.7.10 (\"the\nofficial deprecation date\"), and then start erroring out on \"-b\" three\ncycles after that.\n"},{"id":"186891","messageId":"7vsjhcuz18.fsf_-_@alter.siamese.dyndns.org","threadId":"29913","inReplyTo":"7vipi8wect.fsf@alter.siamese.dyndns.org","subject":"[PATCH] git-am: officially deprecate -b/--binary","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-13T18:38:27Z","receivedAt":"2012-03-13T18:38:27Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"We have had these options as harmless no-op for more than 3 years without\nofficially deprecating them. Let's announce the deprecation and start\nwarning against their use, but without failing the command just not yet,\nso that we can later repurpose the option if we want to in the future.\n\nSigned-off-by: Junio C Hamano \n---\n Documentation/RelNotes/1.7.10.txt | 4 ++--\n git-am.sh | 4 ++--\n 2 files changed, 4 insertions(+), 4 deletions(-)\n\ndiff --git a/Documentation/RelNotes/1.7.10.txt b/Documentation/RelNotes/1.7.10.txt\nindex 65df74b..6286485 100644\n--- a/Documentation/RelNotes/1.7.10.txt\n+++ b/Documentation/RelNotes/1.7.10.txt\n@@ -26,8 +26,8 @@ Compatibility Notes\n Git v1.7.8 or newer.\n \n * The \"--binary/-b\" options to \"git am\" have been a no-op for quite a\n- while and was deprecated in mid 2008 (v1.6.0). When you give these\n- options to \"git am\", it will now fail with an error.\n+ while and were deprecated in mid 2008 (v1.6.0). When you give these\n+ options to \"git am\", it will now warn and ask you not to use them.\n \n \n Updates since v1.7.9\ndiff --git a/git-am.sh b/git-am.sh\nindex faae820..db6ade3 100755\n--- a/git-am.sh\n+++ b/git-am.sh\n@@ -380,8 +380,8 @@ do\n \t-i|--interactive)\n \t\tinteractive=t ;;\n \t-b|--binary)\n-\t\techo >&2 \"The -b/--binary option was deprecated in 1.6.0 and removed in 1.7.10.\"\n-\t\tdie \"Please adjust your scripts.\"\n+\t\techo >&2 \"The $1 option was deprecated in 1.6.0 and will be removed.\"\n+\t\techo >&2 \"Please do not use it anymore.\"\n \t\t;;\n \t-3|--3way)\n \t\tthreeway=t ;;\n"}]}