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

Stephen Gallagher sgallagh at redhat.com
Wed May 22 12:05:35 UTC 2013


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

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.

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.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.13 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iEYEARECAAYFAlGctI4ACgkQeiVVYja6o6MRLACfbhLUyjrMehlc9TYLAIOaaD9Z
PvwAoJFbDZsDREj1K2BPC5A6vR69PYQj
=N5s9
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Fix-typos-and-extra-autoreconf.patch
Type: text/x-patch
Size: 1529 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130522/0acc23e3/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Remove-old-hash-support-from-example-spec.patch
Type: text/x-patch
Size: 811 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130522/0acc23e3/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Fix-typos-and-extra-autoreconf.patch.sig
Type: application/pgp-signature
Size: 72 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130522/0acc23e3/attachment.sig>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Remove-old-hash-support-from-example-spec.patch.sig
Type: application/pgp-signature
Size: 72 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130522/0acc23e3/attachment-0001.sig>


More information about the sssd-devel mailing list