On 2014-10-04 20:12, Antonio Trande wrote:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Hi all.
These warnings appear during a package review; apparently,
fedora-review command completes all its tasks.
WARNING: Illegal return from
/usr/share/fedora-review/scripts/generic-excludearch.sh, code 82,
output: stdout:None stderr:./review-env.sh: line 7: unset:
`BASH_FUNC_module()': not a valid identifier
Hm.... seems that recent bash patch to fix the shellshock problem
introduces this. Fedora-review relies on exported shell functions
(export -f) and the bash fix changes the syntax for exported functions
in an incompatible way.
For now, this means that you should check these things manually:
- Nothing under /srv, /opt or /usr/local
- Package (or it's dependencies) is not known to require an ExcludeArch tag.
- Large data in /usr/share should live in a noarch subpackage.
- Large (or many) docs must go in a -doc subpackage.
Besides this, you can safely ignore these errors.
For fedora-review we need to evaluate what is reqiured to stay
compatible with bash after this Shellshock issue.
Cheers!
--alec
--
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct
