On (22/05/13 08:05), Stephen Gallagher wrote:
On 05/22/2013 04:09 AM, Lukas Slebodnik wrote:
> On (22/05/13 09:16), Sumit Bose wrote:
>> On Wed, May 22, 2013 at 08:24:08AM +0200, Lukas Slebodnik wrote:
>>> ehlo,
>>>
>>> Recommended way to create SRPM is to run make
>>> (prerelease-)srpm. But in previous case make file have to be
>>> generated, therefore configure script should not fail. (all
>>> sssd required dependencies have to be installed) Script
>>> make_srpm.sh can be runned without running configure, script
>>> can be runned only from git repository.
>>>
>>>
https://fedorahosted.org/sssd/ticket/1927
>>>
>>> Patch is attached.
>>
>> Hi Lukas,
>>
>> I haven't tested the script yet, nevertheless I have two
>> comments. Please add a copyright notice and a licence.
>> Additionally I think '--define
> I added copyright.
>
>> _source_filedigest_algorithm=1' should be switch on by an option.
>> Iirc we added it because RHEL5 could not handle newer hash
>> algorithms here. But on newer platforms we should use the
>> platform defaults.
> I removed line with _source_filedigest_algorithm, because sssd
> master no longer compiles on rhel5, nightly builds for rhel5 are
> done from sssd-1-9 branch. But if you think, that it isn't proper
> solution I can add new switch for this.
>
> Thank you very much for comments.
>
> New patch attached.
>
Almost an ack :)
I am attaching two patches, one that should be squashed in and another
that can go in on its own since it's related.
Patch 0001: Fix English grammar and remove one sed line.
There's no reason to substitute autoreconf into the spec file, as it's
already in there. The resulting spec would be calling it twice, which
is wasteful.
I don't know why I thought, that there wasn't autoreconf.
Honestly, I didn't look to sssd.spec.in first time.
Thank you.
I squashed the first patch to my patch.
Patch 0002: Given the argument above regarding
_source_filedigest_algorithm (which is correct), I added a patch to
remove that from the Makefile.am as well.
I am attaching the second patch
unchanged.
LS