Re: [ITP] aide 0.17.3

[Brian Inglis <Brian.Inglis@SystematicSw.ab.ca>]

On 2021-11-12 05:48, Jason Pyeron wrote:
>> -----Original Message-----
>> From: Brian Inglis
>> Sent: Wednesday, November 10, 2021 11:25 AM
>>
>> On 2021-11-10 05:33, Jason Pyeron wrote:
>>> On Sunday, October 31, 2021 10:37 AM, Jason Pyeron wrote:
>>>> On Sunday, October 31, 2021 8:48 AM, Jon Turney wrote:
>>>>> On 29/09/2021 15:27, Jason Pyeron wrote:
>>>>>> On Friday, July 30, 2021 10:34 AM, Jason Pyeron wrote:
>>>>>>> AIDE - Advanced Intrusion Detection Environment
>>>>>>> https://github.com/aide/aide/
>>>>>>> It is a GPL v2 tool for monitoring file system changes.
>>>>>>> There was no (mature?) Windows open source solution until AIDE was built and tested for
>>>>>>> Cygwin. This fills a long standing gap in needs.
>>>>>>> Closed source alternative - Trip Wire.
> ...
>>>>>> Upstream has expressed willingness to review/track patches, if
>>>>>> needed.
>>
>> Good idea to submit patches upstream, as it reduces the number of
>> patches you have to maintain and rebase, and they may have a better idea
>> of how to achieve the same goal with more generality having their
>> knowledge of the package source and build.
>>
> ...
>>>> The attached (with required patch) has been in testing on multiple
>>>> windows servers since late July. They can also be reviewed on
>>>> github [2].
>> Using github is an issue for some: gitlab, bitbucket, etc. may or may
>> not be.
>>
>> That is why it is a good idea to checkout your repo on a playground
>> branch, then force push your repo to:
>>
>> 	ssh://cygwin@cygwin.com/git/cygwin-packages/playground
>>
>> and post the jobs.cgi, run, and log links.
> 
> Run 3524: https://cygwin.com/cgi-bin2/jobs.cgi?id=3524
> 
> x86, x86_64 build logs: https://github.com/cygwin/scallywag/actions/runs/1453082339

To suppress source patch generation at the end of source packaging:

	>>> Creating source patches
	 include/util.h |    1 +
	 1 file changed, 1 insertion(+)

need to add near the end of your definitions:

	DIFF_EXCLUDES=include/util.h

With that change made, LGTM.

>> those out and reports them at the end of your build. You should be
>> seeing those package names duplicated at the end of your cygport ...
>> {package,pkg,{,almost}all}{,-test} run e.g.:
>>
>>   >>> aide requires: cygwin libmhash2 libpcre1 zlib0 cygwin libmhash2
>> libpcre1 zlib0
>>
>> Please also ensure that the package builds cleanly on both arches.
> 
> Done! Thanks for the feedback.
> 
> Great job on the CI infrastructure.

Think that's Jon Turney's great idea and initiative first in Appveyor 
then GH actions.

-- 
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada

This email may be disturbing to some readers as it contains
too much technical detail. Reader discretion is advised.
[Data in binary units and prefixes, physical quantities in SI.]