Bug 235673

Summary: cmake: executable shlibs
Product: [Fedora] Fedora Reporter: Rex Dieter <rdieter>
Component: cmakeAssignee: Orion Poplawski <orion>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: kevin
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 2.4.6-3.fc7 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-04-16 16:38:26 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
executable shlibs none

Description Rex Dieter 2007-04-09 14:56:46 UTC 
Per upstream report:
http://www.cmake.org/Bug/bug.php?op=show&bugid=4805
cmake doesn't make shlibs executable by default, per ChangeLog.txt:
--------------
2006-03-03 18:44  king
...
          Default permissions for shared libraries on
          non-Windows/non-OSX platforms no longer has the execute bit set.
--------------

Attached is a patch to revert that behavior (I think, haven't tested it much)
Comment 1 Rex Dieter 2007-04-09 14:56:49 UTC 
Created attachment 151991 [details]
executable shlibs
Comment 2 Rex Dieter 2007-04-16 15:01:20 UTC 
I'd say go ahead and patch cmake to "just work" (for us), so that the cmake
guideline can be deployed asap without having to mention the shlibs hack.

It looks like cmake upstream will likely do the right thing soon, but I'd rather
not wait for the next cmake release.
Comment 3 Orion Poplawski 2007-04-16 16:38:26 UTC 
Applied the fix from http://public.kitware.com/Bug/bug.php?op=show&bugid=4805
and upstream CVS that uses CMAKE_INSTALL_SO_NO_EXE and makes executable shared
libraries the default except on Debian/Ubuntu.
Comment 4 Rex Dieter 2007-04-16 16:47:14 UTC 
Thanks.


Please port changes to FC-5/FC-6 branches too, and we're golden.
Comment 5 Kevin Kofler 2007-05-01 12:22:53 UTC 
Yuck, this detects the distro from /proc/version. This is going to break the 
day someone wants to build Fedora packages in a mock chroot on a Debian system:
http://packages.debian.org/unstable/devel/mock
(With mock, the kernel and /proc from the host system is used.)

Can the setting be preseeded somehow in the cmake RPM? Or maybe added to the 
command line of the %cmake macro if that works? Or maybe you can get upstream 
to use a saner method to detect the distribution in use, such as the contents 
of /etc/issue or the existence of /etc/debian_version?
Comment 6 Rex Dieter 2007-05-01 12:38:26 UTC 
Kevin, please take your issues upstream, maybe even
http://public.kitware.com/Bug/bug.php?op=show&bugid=4805
they should be made aware of your (imo, valid) reservations.