Re: [gentoo-portage-dev] [Patch] 1/1 New commitmsgfile "cat/pkg: " auto-substitution
On 09/15/2016 03:27 PM, Brian Dolbec wrote: > On Thu, 15 Sep 2016 14:47:31 -0700 > Zac Medicowrote: > >> On 09/15/2016 12:50 PM, Brian Dolbec wrote: >>> >>> From 08f6d22b366c8dc2968573ebc4fd4f0401ea67b7 Mon Sep 17 00:00:00 >>> 2001 From: Brian Dolbec >>> Date: Thu, 15 Sep 2016 12:45:10 -0700 >>> Subject: [PATCH] repoman: Update man page and commitmsgfile option >>> >>> Use .lower() on the leading message 9 charachters of hte message >>> text for the substitution test. >>> Update the man page for this new templating capability. >>> --- >>> repoman/man/repoman.1 | 6 +- >>> repoman/pym/repoman/actions.py | 2 +- >>> 2 files changed, 6 insertions(+), 2 deletions(-) >>> >>> diff --git a/repoman/man/repoman.1 b/repoman/man/repoman.1 >>> index 4a9122e..8df3207 100644 >>> --- a/repoman/man/repoman.1 >>> +++ b/repoman/man/repoman.1 >>> @@ -109,7 +109,11 @@ Behave as if no package.mask entries exist >>> (not allowed with commit mode) Adds a commit message via the >>> command line .TP >>> \fB-M\fR, \fB--commitmsgfile\fR >>> -Adds a commit message from the specified file >>> +Adds a commit message from the specified file. This option also >>> will perform +an automatic text substitution of a leading "cat/pkg: >>> " string (upper or lower +case) with the actual category/package >>> prefix as defined by the required message +format. Use this option >>> for templating a common commit message for multiple +package >>> updates. .TP >>> \fB-V\fR, \fB--version\fR >>> Show version info >>> diff --git a/repoman/pym/repoman/actions.py >>> b/repoman/pym/repoman/actions.py index 0534c29..9763c26 100644 >>> --- a/repoman/pym/repoman/actions.py >>> +++ b/repoman/pym/repoman/actions.py >>> @@ -108,7 +108,7 @@ class Actions(object): >>> " >>> --commitmsgfile='%s'\n" % self.options.commitmsgfile) else: >>> raise >>> - if commitmessage[:9] in ("cat/pkg: ", >>> "CAT/PKG: "): >>> + if commitmessage[:9].lower() in ("cat/pkg: >>> "): >> >> This is a string-in-string containment check unless you add a comma to >> make it a tuple like ("cat/pkg: ",). >> > > Doh, I removed that comma without realizing, thanks Otherwise, looks good. -- Thanks, Zac
Re: [gentoo-portage-dev] [Patch] 1/1 New commitmsgfile "cat/pkg: " auto-substitution
On Thu, 15 Sep 2016 14:47:31 -0700 Zac Medicowrote: > On 09/15/2016 12:50 PM, Brian Dolbec wrote: > > > > From 08f6d22b366c8dc2968573ebc4fd4f0401ea67b7 Mon Sep 17 00:00:00 > > 2001 From: Brian Dolbec > > Date: Thu, 15 Sep 2016 12:45:10 -0700 > > Subject: [PATCH] repoman: Update man page and commitmsgfile option > > > > Use .lower() on the leading message 9 charachters of hte message > > text for the substitution test. > > Update the man page for this new templating capability. > > --- > > repoman/man/repoman.1 | 6 +- > > repoman/pym/repoman/actions.py | 2 +- > > 2 files changed, 6 insertions(+), 2 deletions(-) > > > > diff --git a/repoman/man/repoman.1 b/repoman/man/repoman.1 > > index 4a9122e..8df3207 100644 > > --- a/repoman/man/repoman.1 > > +++ b/repoman/man/repoman.1 > > @@ -109,7 +109,11 @@ Behave as if no package.mask entries exist > > (not allowed with commit mode) Adds a commit message via the > > command line .TP > > \fB-M\fR, \fB--commitmsgfile\fR > > -Adds a commit message from the specified file > > +Adds a commit message from the specified file. This option also > > will perform +an automatic text substitution of a leading "cat/pkg: > > " string (upper or lower +case) with the actual category/package > > prefix as defined by the required message +format. Use this option > > for templating a common commit message for multiple +package > > updates. .TP > > \fB-V\fR, \fB--version\fR > > Show version info > > diff --git a/repoman/pym/repoman/actions.py > > b/repoman/pym/repoman/actions.py index 0534c29..9763c26 100644 > > --- a/repoman/pym/repoman/actions.py > > +++ b/repoman/pym/repoman/actions.py > > @@ -108,7 +108,7 @@ class Actions(object): > > " > > --commitmsgfile='%s'\n" % self.options.commitmsgfile) else: > > raise > > - if commitmessage[:9] in ("cat/pkg: ", > > "CAT/PKG: "): > > + if commitmessage[:9].lower() in ("cat/pkg: > > "): > > This is a string-in-string containment check unless you add a comma to > make it a tuple like ("cat/pkg: ",). > Doh, I removed that comma without realizing, thanks -- Brian Dolbec
Re: [gentoo-portage-dev] [Patch] 1/1 New commitmsgfile "cat/pkg: " auto-substitution
On 09/15/2016 12:50 PM, Brian Dolbec wrote: > > From 08f6d22b366c8dc2968573ebc4fd4f0401ea67b7 Mon Sep 17 00:00:00 2001 > From: Brian Dolbec> Date: Thu, 15 Sep 2016 12:45:10 -0700 > Subject: [PATCH] repoman: Update man page and commitmsgfile option > > Use .lower() on the leading message 9 charachters of hte message text for the > substitution > test. > Update the man page for this new templating capability. > --- > repoman/man/repoman.1 | 6 +- > repoman/pym/repoman/actions.py | 2 +- > 2 files changed, 6 insertions(+), 2 deletions(-) > > diff --git a/repoman/man/repoman.1 b/repoman/man/repoman.1 > index 4a9122e..8df3207 100644 > --- a/repoman/man/repoman.1 > +++ b/repoman/man/repoman.1 > @@ -109,7 +109,11 @@ Behave as if no package.mask entries exist (not allowed > with commit mode) > Adds a commit message via the command line > .TP > \fB-M\fR, \fB--commitmsgfile\fR > -Adds a commit message from the specified file > +Adds a commit message from the specified file. This option also will perform > +an automatic text substitution of a leading "cat/pkg: " string (upper or > lower > +case) with the actual category/package prefix as defined by the required > message > +format. Use this option for templating a common commit message for > multiple +package updates. > .TP > \fB-V\fR, \fB--version\fR > Show version info > diff --git a/repoman/pym/repoman/actions.py > b/repoman/pym/repoman/actions.py index 0534c29..9763c26 100644 > --- a/repoman/pym/repoman/actions.py > +++ b/repoman/pym/repoman/actions.py > @@ -108,7 +108,7 @@ class Actions(object): > " > --commitmsgfile='%s'\n" % self.options.commitmsgfile) else: > raise > - if commitmessage[:9] in ("cat/pkg: ", > "CAT/PKG: "): > + if commitmessage[:9].lower() in ("cat/pkg: "): This is a string-in-string containment check unless you add a comma to make it a tuple like ("cat/pkg: ",). > commitmessage = self.msg_prefix() + > commitmessage[9:] > if not commitmessage or not commitmessage.strip(): > -- Thanks, Zac
Re: [gentoo-portage-dev] [Patch] 1/1 New commitmsgfile "cat/pkg: " auto-substitution
On Thu, 15 Sep 2016 12:50:13 -0700 Brian Dolbecwrote: > From 08f6d22b366c8dc2968573ebc4fd4f0401ea67b7 Mon Sep 17 00:00:00 2001 > From: Brian Dolbec > Date: Thu, 15 Sep 2016 12:45:10 -0700 > Subject: [PATCH] repoman: Update man page and commitmsgfile option > > Use .lower() on the leading message 9 charachters of hte message text > for the substitution test. Fixed this line: Use .lower() on the leading 9 characters of the message text for the substitution test. -- Brian Dolbec