Git Product home page Git Product logo

Comments (9)

voxik avatar voxik commented on July 23, 2024

I'd submit fix for this, but the question is what is the real intent? What is considered stable?

Given that the comment says: "Built-in ruby lib stuff doesn't change, but things can occasionally be installed into sitedir, which generally lives under libdir.", then there should probably be matched libdir as well as datadir.

Another option could be to match rubylibdir instead. That way, the exception for sitedir could be probably dropped, not sure.

BTW this issue is reproducible on Fedora.

from bootsnap.

casperisfine avatar casperisfine commented on July 23, 2024

Thanks for the report. I'll need to dig into the history a bit to see what this stable thing is about.

from bootsnap.

casperisfine avatar casperisfine commented on July 23, 2024

but the question is what is the real intent? What is considered stable?

Ok, so "stable" here is to describe code that come from either the stdlib or gems, as opposed to your application code.

So anything coming from the ruby installation can be considered stable.

I'd submit fix for this

Please do if the above clarified your question.

from bootsnap.

voxik avatar voxik commented on July 23, 2024

The code currently assumes that everything, what is under /usr/lib64 is stable, with possible exception to sitedir. On standard distribution, that might include large variety of files, i.e. majority of the distribution. If that was the intention, then this should also include the /usr/share.

But I'm going to assume, that intention was to include just Ruby StdLib.

from bootsnap.

voxik avatar voxik commented on July 23, 2024

Also, I am not Bootsnap expert, so the question is, if only Ruby files needs to be covered or if also arch specific files (.so libraries) should be included.

from bootsnap.

voxik avatar voxik commented on July 23, 2024

I'm going assume the former.

from bootsnap.

casperisfine avatar casperisfine commented on July 23, 2024

Yeah .so files should be included.

from bootsnap.

voxik avatar voxik commented on July 23, 2024

Yeah .so files should be included.

Oh, Ok, going to fix the PR to include them.

from bootsnap.

casperisfine avatar casperisfine commented on July 23, 2024

Should be fixed by 72202aa, let me know if that's not the case.

from bootsnap.

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.