[Pkg-nginx-maintainers] Bug#869499: Bug#869499: nginx: [PATCH] Automate modules watch & upgrade process

Mpampis Kostas charkost.rb at gmail.com
Tue Nov 7 15:54:32 UTC 2017


Hello Christos,

Attached a new patch version. It contains the following changes:

* Rebase to current origin/master with cannonical module paths.
* Rename debian/modules/README.Modules-version to debian/modules/control.
* Convert the uscan to a more generic command named ngxmod with uscan as 
a subcommand.
* Auto bump module version in debian/modules/control after an upgrade.
* Auto commit the module source changes after an upgrade.


On 08/31/2017 04:08 PM, Christos Trochalakis wrote:
> On Sun, Jul 23, 2017 at 09:42:47PM +0300, Mpampis Kostas wrote:
>> Package: nginx
>> Severity: wishlist
>> Tags: patch
>>
>> This patch introduces the debian/modules/uscan script and some helper
>> files which can be used to automate the modules watch & upgrade process.
>>
>> The main script is executed as '$ debian/modules/uscan' without any 
>> arguments from
>> the root of the debianized nginx source tree where you see the 
>> debian/ directory.
>>
>> It runs uscan for each nginx module listed in the newly deb822 formatted
>> debian/modules/README.Modules-version using the watchfiles in 
>> debian/modules/watch.
>>
>> If a new version is available, it will ask you if you want to upgrade.
>> If you agree, it will download the tarball, place it in the parent 
>> directory
>> and upgrade the module's source.
>>
>> After the upgrade, it will remove the files listed in the
>> Files-Excluded module field.
>>
>> Example output:
>>
>> $ debian/modules/uscan
>
> Thanks a lot Mpampis, great patch, it's a time saver!
>
> Since the module config is now a proper deb822 file, we can rename it to
> debian/modules/control. As a final step, this might be a good time to
> cannonicalize the module paths to exactly match the package naming. I
> believe this should happen first to keep the diffs as simple as
> possible.
>
> Regarding the script itself, I believe we can make it more generic
> (ngxmod?) and add uscan as a subcommand, we can the add more commands
> like list, add, etc.
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Automate-modules-watch-upgrade-process.patch
Type: text/x-patch
Size: 17684 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-nginx-maintainers/attachments/20171107/1b7998dc/attachment.bin>


More information about the Pkg-nginx-maintainers mailing list