Red Hat Bugzilla – Bug 431539
rpm build errors, and bad spec file coding, and bad configure coding, mostly involving SELINUX
Last modified: 2008-03-04 07:40:48 EST
Description of problem:
There are a number of problems with 'at' spec file and also configure.in
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. Read at.spec
2. Read configure.in
1. Externally defined WITH_SELINUX will always be ignored (eg from
/etc/rpm/macros or from build command-line), unlike other variables like
WITH_PAM. Why is the spec file coded to properly handle WITH_PAM but not
similarly for WITH_SELINUX? Is this simply a form of coersion, because that is
the only way to gain acceptance of SELINUX by the community?
2. Handling of --with-selinux in configure.in is coded so that absence of
--with-selinux does not work as expected. What is the point of having options
like --with-selinux if it is just a no-op?
1. spec files should be written properly .. perhaps somebody from Redhat could
give high quality guidance here.
2. configure.in should be written properly. The handling of --with-FEATURE is
normally done properly except for when SELINUX is involved (for some mysterious
reason). There are plenty of examples of proper FEATURE handling, so there is
no excuse for not getting simple things like this right ... unless one has
ulterior motives in trying to cripple peoples ability to disable the pointless
and hysterical/paranoid SELINUX feature.
Disabling SELinux and supporting custom builds with random features
enabled/disabled is definitely not a priority. So feel free to provide patches
if you want this fixed.
If only the person who added the selinux stuff had just done it properly in the
first place then it would have saved everybody, especially myself, a lot of bother.
But why bother added options like --with-selinux which don't work? Shouldn't
adding those options (not working at that) be an even lower priority?
Also, I see this happening over and over again in nearly every package created
by RedHat employees. The same mistakes keep getting reproduced. Why not just do
it right all of the time? I really is so very simple. After all RedHat
invented rpm so surely they should be the best at getting rpms right?
The fix of configure isn't visible fix for those who don't use selinux. If you
are so interested in clean code, then please attach the patch for review.
I don't use selinux and the fix of configure is visible.
And don't get me wrong - I don't like clean code either!
I'm really appreciating your concern is the base packages, but I definitely
won't rewrite whole configure and Makefile according to new auto tools.
It could be closed as WONTFIX or UPSTREAM.