Comments (10)
PRs welcome -- if you're game to give it a try, we'd be happy to help you finish off even a half-working attempt.
Sure, will give it a try when I am free.
from brew.
But we should try fixing this for
brew edit
. PRs welcome -- if you're game to give it a try, we'd be happy to help you finish off even a half-working attempt.
Agreed. Please scope this logic entirely into dev-cmd/edit.rb
if so, thanks!
from brew.
Original issue analysis is incorrect as the sandbox and build.rb
doesn't apply here.
But there is overall environment filtering going on and setting HOMEBREW_TEMP
does work: HOMEBREW_TEMP=$TMPDIR brew edit karabiner-elements
. This perhaps should happen by default for brew edit
but currently doesn't.
from brew.
Original issue analysis is incorrect as the sandbox and
build.rb
doesn't apply here.
Thank you for pointing this out.
But there is overall environment filtering going on and setting HOMEBREW_TEMP does work: HOMEBREW_TEMP=$TMPDIR brew edit karabiner-elements. This perhaps should happen by default for brew edit but currently doesn't.
Is there any side effect if the user sets HOMEBREW_TEMP
to TMPDIR
globally? Or do I need to wait for changes from brew edit
?
from brew.
Is there any side effect if the user sets
HOMEBREW_TEMP
toTMPDIR
globally?
It could break some builds that get confused by the long paths that TMPDIR
typically point to.
If you don't build anything from source you'll probably be fine. If you do encounter build failures, you can always try to unset HOMEBREW_TEMP
.
from brew.
Thank you for your reply ;), then I will temporarily leave this issue open. Feel free to close it if your team thinks it should not be implemented for brew edit
.
from brew.
You could probably also set
TMPDIR=/private/tmp
as a likely safer alternative (though I don't think HOMEBREW_TEMP=$TMPDIR
is that unsafe either).
But we should try fixing this for brew edit
. PRs welcome -- if you're game to give it a try, we'd be happy to help you finish off even a half-working attempt.
from brew.
Hi, after a rough reading of the edit command code I have some doubts.
I don't quite understand why HOMEBREW_TEMP=$TMPDIR brew edit karabiner-elements
can work. How does setting HOMEBREW_TEMP
affect the access to the sock file in TMPDIR
by exec emacsclient
? Is it because TMPDIR
is reset to HOMEBREW_TEMP
during command execution (not found in the code) or is there another reason?
from brew.
Is it because
TMPDIR
is reset toHOMEBREW_TEMP
during command execution (not found in the code)
Yes, this is done by bin/brew
.
from brew.
Is it because
TMPDIR
is reset toHOMEBREW_TEMP
during command execution (not found in the code)Yes, this is done by
bin/brew
.
But bin/brew
is just filter TMPDIR
and I miss up the logic that sets TMPDIR
back to HOMEBREW_TEMP
in dev-cmd/edit.rb
or other places like
brew/Library/Homebrew/utils/gems.rb
Lines 99 to 101 in ff950e6
HOMEBREW_TEMP=$TMPDIR brew edit karabiner-elements
work.
Update: In fact gems.rb
is run before edit.rb
, so the $TMPDIR is set to HOMEBREW_TEMP
there.
from brew.
Related Issues (20)
- Throttle does not work for llama.cpp HOT 2
- Support specifying replacement formulae in `deprecate!` or `disable!` calls HOT 1
- Brew appears to be broken HOT 3
- shellenv.sh: Failure when user cannot write "${HOMEBREW_PATHS_FILE}" HOT 1
- Archive and deprecate HOT 1
- Consider setting `XDG_DATA_DIRS` in `shellenv` on Linux HOT 1
- Multiple Formula giving `Error: Couldn't find manifest matching bottle checksum.` HOT 4
- pkg missing for release 4.3.21 assets HOT 4
- sha256 checksum is not enforced HOT 2
- Setup `brew determine-test-runners` for macOS 15 x86_64 HOT 11
- Avoid outputting Rosetta 2 caveats on Intel machines HOT 15
- Don't allow installing formulae from paths without HOMEBREW_DEVELOPER HOT 2
- `brew install` should display download size HOT 18
- `brew install [email protected]` failed while exec `go build` with custom system env `GOPROXY=https://goproxy.io,direct` HOT 1
- Make `brew tap-info` show commit hash for taps that don't use the JSON API (_i.e._, are cloned locally) HOT 5
- Using `actionlint` in `brew style` makes updating `actionlint` impossible HOT 6
- Piping password to brew no longer working HOT 7
- NOT support for macOS 15 HOT 3
- could you update update the hombrew versioin to suppourt macos 15 HOT 1
- postgresql@16 service can only be started by sudo HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
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.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from brew.