Skip to content

install: fetch new tags/branches when updating a cached git dependency #35566

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
robobun wants to merge 5 commits into from
Closed

install: fetch new tags/branches when updating a cached git dependency #35566

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
2549639
install: fetch new tags/branches when updating a cached git dependency
robobun Aug 1, 2026
5c4fefb
Merge remote-tracking branch 'origin/main' into farm/1fe6d090/git-fet…
robobun Aug 13, 2026
d15b182
install: cover bun update and the lockfile pin in the git cache refre…
robobun Aug 13, 2026
f190df1
install: fetch origin with --prune and clone with -o origin; fold ref…
robobun Aug 13, 2026
b15ff46
test: point GIT_CONFIG_GLOBAL at an empty file so the global gitconfi…
robobun Aug 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 18 additions & 1 deletion src/install/repository.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -722,7 +722,21 @@ impl RepositoryExt for Repository {
&[folder_name.as_bytes()],
);

if let Err(err) = exec(env, &[b"git", b"-C", path, b"fetch", b"--quiet"]) {
// Bare clones have no fetch refspec; --prune clears refs renamed upstream.
if let Err(err) = exec(
env,
&[
b"git",
b"-C",
path,
b"fetch",
b"--quiet",
b"--prune",
b"origin",
Comment thread
claude[bot] marked this conversation as resolved.
Resolved
b"+refs/heads/*:refs/heads/*",
Comment thread
robobun marked this conversation as resolved.
Resolved
b"+refs/tags/*:refs/tags/*",
],
) {
Comment thread
robobun marked this conversation as resolved.
Resolved
log.add_error_fmt(
None,
bun_ast::Loc::EMPTY,
Expand Down Expand Up @@ -751,6 +765,9 @@ impl RepositoryExt for Repository {
b"core.longpaths=true",
b"--quiet",
b"--bare",
// The refresh fetches "origin" by name; ignore clone.defaultRemoteName.
b"-o",
b"origin",
url,
target,
],
Expand Down
Loading