Re: [Rpm-maint] [rpm-software-management/rpm] python-macro-helper should not have shebang and x flag (#387)

2018-02-12 Thread Panu Matilainen
Fixed in commit faf7d87fda3456382cf161ff11a2ad8e00ae4ab0, thanks for the report

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/387#issuecomment-364898947___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] python-macro-helper should not have shebang and x flag (#387)

2018-01-31 Thread Miro Hrončok
Also, this makes the rpmb-build package in fedora require python2: 
https://bugzilla.redhat.com/show_bug.cgi?id=1538657

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/387#issuecomment-361934178___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


[Rpm-maint] [rpm-software-management/rpm] python-macro-helper should not have shebang and x flag (#387)

2018-01-30 Thread Miro Hrončok
Looking at 7ac3f07ac41a2760ed6dae65e63fe0b49aec6f6c the 
`%{_rpmconfigdir}/python-macro-helper` file is always invoked with 
`%{__python}` and thus doesn't need a shebang and an executable flag. Since 
`%{__python}` can be redefined, having a hardcoded  `/usr/bin/python` shebang 
that is never used is misleading.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/387___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint