Re: Trailing comments in broken-out series file break quilt
Jean wrote: > You should simply be using an up-to-date version of quilt, namely > version 0.42, which supports Andrew-style comments in series files just > fine. Right you are - that works too. Thanks for the good work on quilt and thanks for pointing this out. -- I won't rest till it's the best ... Programmer, Linux Scalability Paul Jackson <[EMAIL PROTECTED]> 1.925.600.0401 - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Trailing comments in broken-out series file break quilt
Hi Paul, > However the quilt command passes these additional terms to the patch > command as additional arguments, confusing the heck out of patch, > and generating an error message that confused the heck out of me. > > Question - should I be asking Andrew not to comment this way, or > should I be asking quilt to recognize a comment convention here? You should simply be using an up-to-date version of quilt, namely version 0.42, which supports Andrew-style comments in series files just fine. Thanks, -- Jean Delvare - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Trailing comments in broken-out series file break quilt
Andrew wrote: > I'll just stop using them. That works - thanks. -- I won't rest till it's the best ... Programmer, Linux Scalability Paul Jackson <[EMAIL PROTECTED]> 1.925.600.0401 - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Trailing comments in broken-out series file break quilt
Andrew wrote: I'll just stop using them. That works - thanks. -- I won't rest till it's the best ... Programmer, Linux Scalability Paul Jackson [EMAIL PROTECTED] 1.925.600.0401 - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Trailing comments in broken-out series file break quilt
Hi Paul, However the quilt command passes these additional terms to the patch command as additional arguments, confusing the heck out of patch, and generating an error message that confused the heck out of me. Question - should I be asking Andrew not to comment this way, or should I be asking quilt to recognize a comment convention here? You should simply be using an up-to-date version of quilt, namely version 0.42, which supports Andrew-style comments in series files just fine. Thanks, -- Jean Delvare - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Trailing comments in broken-out series file break quilt
Jean wrote: You should simply be using an up-to-date version of quilt, namely version 0.42, which supports Andrew-style comments in series files just fine. Right you are - that works too. Thanks for the good work on quilt and thanks for pointing this out. -- I won't rest till it's the best ... Programmer, Linux Scalability Paul Jackson [EMAIL PROTECTED] 1.925.600.0401 - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Trailing comments in broken-out series file break quilt
Paul Jackson <[EMAIL PROTECTED]> wrote: > > md-fix-rh_dec-rh_inc-race-in-dm-raid1c.patch # wait > > ... > Question - should I be asking Andrew not to comment this way, or > should I be asking quilt to recognize a comment convention here? I'll just stop using them. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Trailing comments in broken-out series file break quilt
Apparently Andrews patch tools allow trailing comments on active lines in the series file, as in these lines culled from the series file for 2.6.13-rc6-mm2: e1000-numa-aware-allocation-of-descriptors-v2.patch # hold nfs-nfs3-page-null-fill-in-a-short-read-situation.patch # wait sched-implement-nice-support-across-physical-cpus-on-smp.patch # con sched-change_prio_bias_only_if_queued.patch # con sched-account_rt_tasks_in_prio_bias.patch # con sched-smp-nice-bias-busy-queues-on-idle-rebalance.patch # con sched-correct_smp_nice_bias.patch # con md-fix-rh_dec-rh_inc-race-in-dm-raid1c.patch # wait However the quilt command passes these additional terms to the patch command as additional arguments, confusing the heck out of patch, and generating an error message that confused the heck out of me. Question - should I be asking Andrew not to comment this way, or should I be asking quilt to recognize a comment convention here? If we choose the second alternative, then the following change to the file /usr/local/share/quilt/scripts/patchfns might to the trick: --- /tmp/q/patchfns.1 2005-08-29 19:11:24.0 -0700 +++ /tmp/q/patchfns.2 2005-08-29 19:11:31.0 -0700 @@ -108,8 +108,8 @@ patch_args() then /bin/gawk ' $1 == "'"$patch"'" \ - { if (NF >= 2) - for (i=2; i <= NF; i++) + { if (NF >= 2 && $2 != "#") + for (i=2; i <= NF && $i != "#"; i++) print $i else print "-p1" ; -- I won't rest till it's the best ... Programmer, Linux Scalability Paul Jackson <[EMAIL PROTECTED]> 1.925.600.0401 - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Trailing comments in broken-out series file break quilt
Apparently Andrews patch tools allow trailing comments on active lines in the series file, as in these lines culled from the series file for 2.6.13-rc6-mm2: e1000-numa-aware-allocation-of-descriptors-v2.patch # hold nfs-nfs3-page-null-fill-in-a-short-read-situation.patch # wait sched-implement-nice-support-across-physical-cpus-on-smp.patch # con sched-change_prio_bias_only_if_queued.patch # con sched-account_rt_tasks_in_prio_bias.patch # con sched-smp-nice-bias-busy-queues-on-idle-rebalance.patch # con sched-correct_smp_nice_bias.patch # con md-fix-rh_dec-rh_inc-race-in-dm-raid1c.patch # wait However the quilt command passes these additional terms to the patch command as additional arguments, confusing the heck out of patch, and generating an error message that confused the heck out of me. Question - should I be asking Andrew not to comment this way, or should I be asking quilt to recognize a comment convention here? If we choose the second alternative, then the following change to the file /usr/local/share/quilt/scripts/patchfns might to the trick: --- /tmp/q/patchfns.1 2005-08-29 19:11:24.0 -0700 +++ /tmp/q/patchfns.2 2005-08-29 19:11:31.0 -0700 @@ -108,8 +108,8 @@ patch_args() then /bin/gawk ' $1 == '$patch' \ - { if (NF = 2) - for (i=2; i = NF; i++) + { if (NF = 2 $2 != #) + for (i=2; i = NF $i != #; i++) print $i else print -p1 ; -- I won't rest till it's the best ... Programmer, Linux Scalability Paul Jackson [EMAIL PROTECTED] 1.925.600.0401 - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Trailing comments in broken-out series file break quilt
Paul Jackson [EMAIL PROTECTED] wrote: md-fix-rh_dec-rh_inc-race-in-dm-raid1c.patch # wait ... Question - should I be asking Andrew not to comment this way, or should I be asking quilt to recognize a comment convention here? I'll just stop using them. - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/