Giter VIP home page Giter VIP logo

Comments (10)

MikeMcQuaid avatar MikeMcQuaid commented on May 23, 2024

Arguably HOMEBREW_CACHE should just have a different default on Linux.

from legacy-linuxbrew.

sjackman avatar sjackman commented on May 23, 2024

I don't believe there's a suitable world-writable cache on Linux, so I'll change the default to ~/.local/homebrew.

from legacy-linuxbrew.

MikeMcQuaid avatar MikeMcQuaid commented on May 23, 2024

There is a single base directory relative to which user-specific non-essential (cached) data should be written. This directory is defined by the environment variable $XDG_CACHE_HOME.
$XDG_CACHE_HOME defines the base directory relative to which user specific non-essential data files should be stored. If $XDG_CACHE_HOME is either not set or empty, a default equal to $HOME/.cache should be used.
http://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html

I'd suggest using the env if it's there otherwise defaulting to ~/.cache/homebrew

from legacy-linuxbrew.

sjackman avatar sjackman commented on May 23, 2024

Yes, I agree. Does this patch look like something that could be incorporated into mxcl/homebrew?

from legacy-linuxbrew.

sjackman avatar sjackman commented on May 23, 2024

Oh, I missed the quoted part of your comment. Oddly, $XDG_CACHE_HOME is not set on Ubuntu. Does anyone know of a Linux (or other) system that uses these environment variables?

from legacy-linuxbrew.

MikeMcQuaid avatar MikeMcQuaid commented on May 23, 2024

Typically they are user or environment set. Worth respecting them as they are so easy to do.

from legacy-linuxbrew.

MikeMcQuaid avatar MikeMcQuaid commented on May 23, 2024

This is something for the wrapper script to do I think.

from legacy-linuxbrew.

sjackman avatar sjackman commented on May 23, 2024

If you mean the brew shell script, I'm inclined to keep that as simple as possible. I do think though that there's a good argument for factoring out os/linux/global.rb.

from legacy-linuxbrew.

MikeMcQuaid avatar MikeMcQuaid commented on May 23, 2024

Well, I think we'd allow these type of things to be user customisable with environment variables so if you implemented that the wrapper script could simply set its own defaults. Regardless the if/else approach won't really work unfortunately for getting merged up; linux/global may do.

from legacy-linuxbrew.

sjackman avatar sjackman commented on May 23, 2024

Okay. I'll add it to my os/linux todo list. There's other patches in Linuxbrew that should move there as well.

from legacy-linuxbrew.

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.