https://bugzilla.redhat.com/show_bug.cgi?id=1070171



--- Comment #2 from Fabian Affolter <m...@fabian-affolter.ch> ---
Thanks for the review, Christopher.

(In reply to Christopher Meng from comment #1)
> 1. #!/usr/bin/env bash
> 
> I'm not sure if we need to hack it to /bin/bash or /bin/sh.

As far as I remember was this topic discussed 4-5 years ago. rpmbuild is
picking env up so it will work.

> 2. install with -p.

fixed

> 3. Mix using tab and space:

fixed

Updated files:
Spec URL: http://fab.fedorapeople.org/packages/SRPMS/tlssled.spec
SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/tlssled-1.3-2.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to