Bug 432004 - cannot build without selinux
Summary: cannot build without selinux
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: tar
Version: 8
Hardware: All
OS: Linux
low
high
Target Milestone: ---
Assignee: Radek Brich
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Keywords: Reopened
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-02-08 12:05 UTC by JW
Modified: 2008-03-19 12:36 UTC (History)
1 user (show)

(edit)
Clone Of:
(edit)
Last Closed: 2008-03-19 12:36:05 UTC


Attachments (Terms of Use)

Description JW 2008-02-08 12:05:17 UTC
Description of problem:
if selinux is installed on build system, then there is no option to disable selinux

Version-Release number of selected component (if applicable):
tar-1.17-3

How reproducible:
Always

Steps to Reproduce:
1. read configure.ac
2. read tar.spec

  
Actual results:
no option to disable selinux

Expected results:
should contain "--with-selinux" and "--without-selinux" options

Additional info:
RH, you will end up like SCO if you keep going down this path!!

Comment 1 Radek Brich 2008-02-12 14:41:50 UTC
I don't understand. There is a runtime option to enable storing of SELinux
attributes and these attributes are ignored by default. SELinux-enabled tar
should not do any harm if the option is not given.

So, why should anyone need to build alternative tar without SELinux?
If there is some specific problem with SELinux support, please report the bug.
Thank you.

Closing as NOTABUG...

Comment 2 JW 2008-02-12 21:35:02 UTC
What harm would be done by making linkage with SElinux optional?  Instead of
just depending on runtime library detection.

Many other packages do the right thing.  What is so special about tar that it
needs to implement a half-baked selinux configure?

Why not implement --with-selinux?  What is the problem with doing that?




Comment 3 Will Woods 2008-02-12 21:52:35 UTC
SELinux code is not present in upstream tar. It's all added by
tar-1.17-xattrs-conf.patch and tar-1.17-xattrs.patch. If you want to build
without SELinux support, don't apply those patches. 

Comment 4 Radek Brich 2008-03-19 12:36:05 UTC
(In reply to comment #3)
> SELinux code is not present in upstream tar. It's all added by
> tar-1.17-xattrs-conf.patch and tar-1.17-xattrs.patch. If you want to build
> without SELinux support, don't apply those patches. 

Yes, no need for new configure option until the patches are in upstream.

Closing this again...


Note You need to log in before you can comment on or make changes to this bug.