[SSSD] [PATCH] Adding script to create a SRPM

Stephen Gallagher sgallagh at redhat.com
Wed May 22 12:44:31 UTC 2013


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 05/22/2013 08:35 AM, Lukas Slebodnik wrote:
> 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
> 

Ack from me, although someone else should probably approve Patch 0002.


-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.13 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iEYEARECAAYFAlGcva8ACgkQeiVVYja6o6NQoACfQtuJotANebMLxsHr5LgVhRMz
JwgAoJzZws0R7L2RQdTkGdO50OrxZgEu
=PlS2
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list