Giter Site home page Giter Site logo

Comments (7)

tkanteck avatar tkanteck commented on July 17, 2024

Thanks for the contact and the feedback. This is true that 'ldoconfig' is called by lib/Makefile but as part of install target only. Consequently it is not required to be root for library compilation and any user should be able to do it.
We are open to suggestions and pull requests with regard to improvements in the existing build system. We'll also have a look into evil autoconf :)

from intel-cmt-cat.

fenrus75 avatar fenrus75 commented on July 17, 2024

https://github.com/clearlinux-pkgs/intel-cmt-cat/blob/master/build.patch

this is the kind of thing linux distros need to do for their packaging; even make install into a chroot-like directory (with DESTDIR) will be done as non-root.

intel-cmt-cat is pretty much the only package in the whole distro that runs ldconfig as part of "make install".... it's at least highly unusual

from intel-cmt-cat.

tkanteck avatar tkanteck commented on July 17, 2024

How does clearlinux intel-cmt-cat spec file look like? 'make install' is not part of Fedora spec file for instance and so no patching is required.

from intel-cmt-cat.

fenrus75 avatar fenrus75 commented on July 17, 2024

https://github.com/clearlinux-pkgs/intel-cmt-cat/blob/master/intel-cmt-cat.spec

Fedora hates your make install so much that they decided to open code it instead, and put a nasty comment instead... I wouldn't take Fedora's workaround as a sign that the Makefile is doing the right thing ;-)

(from their spec)

Not doing make install as it strips the symbols.

Using files from the build directory.

from intel-cmt-cat.

tkanteck avatar tkanteck commented on July 17, 2024

Thank you but actually we wrote this spec file file so we cannot really hate ourselves.

We'll consider removing ldconfig from the lib/Makefile or creating separate target for packaging.
The only reason 'ldocnfig' is there is for users installing software from a tarball.

from intel-cmt-cat.

fenrus75 avatar fenrus75 commented on July 17, 2024

fwiw Debian seems to have a patch very similar

http://http.debian.net/debian/pool/main/i/intel-cmt-cat/intel-cmt-cat_1.0.1-1.debian.tar.xz

as does opensuse

https://build.opensuse.org/package/view_file/server:monitoring/intel-cmt-cat/fix-ldconfig-call.patch?expand=1

from intel-cmt-cat.

mdcornu avatar mdcornu commented on July 17, 2024

Closing this issue after merging workaround to install without running ldconfig.
Please reopen if needed.

Regards,
Marcel

from intel-cmt-cat.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.