Skip to content

Fix #14687 (Move repository from danmar to cppcheck-opensource)#8486

Open
danmar wants to merge 2 commits intocppcheck-opensource:mainfrom
cppchecksolutions:fix-14687
Open

Fix #14687 (Move repository from danmar to cppcheck-opensource)#8486
danmar wants to merge 2 commits intocppcheck-opensource:mainfrom
cppchecksolutions:fix-14687

Conversation

@danmar
Copy link
Copy Markdown
Collaborator

@danmar danmar commented Apr 24, 2026

No description provided.

@danmar danmar requested a review from firewave April 24, 2026 15:15
@firewave
Copy link
Copy Markdown
Collaborator

Err - how do you move forks if the upstream repo moves?

Please update the daca client version.

And I assume this will wreck havoc on daca for quite a while since all clients need to be updated otherwise they will pull outdated code. And we need to make sure the transition within daca just works if the repo changes.

Any other surprises like this happening soon?

@firewave
Copy link
Copy Markdown
Collaborator

And there's also the external integrations in OSS-Fuzz, Codecov and Coverity which need to be checked.

@danmar
Copy link
Copy Markdown
Collaborator Author

danmar commented Apr 24, 2026

Err - how do you move forks if the upstream repo moves?

personally I used these commands:

git remote remove upstream
git remote add upstream git@github.com:cppchek-opensource/cppcheck.git

And there's also the external integrations in OSS-Fuzz, Codecov and Coverity which need to be checked.

I have updated Codecov and Coverity integrations as far as I know.

@danmar
Copy link
Copy Markdown
Collaborator Author

danmar commented Apr 24, 2026

Any other surprises like this happening soon?

No.

@chrchr-github
Copy link
Copy Markdown
Collaborator

Err - how do you move forks if the upstream repo moves?

There seems to be some kind of redirect, I didn't have to make any changes locally.

Please update the daca client version.

And I assume this will wreck havoc on daca for quite a while since all clients need to be updated otherwise they will pull outdated code. And we need to make sure the transition within daca just works if the repo changes.

daca has been mostly down anyway in recent weeks, so we would hardly notice.

@danmar
Copy link
Copy Markdown
Collaborator Author

danmar commented Apr 24, 2026

oss-fuzz PR: google/oss-fuzz#15410

@danmar
Copy link
Copy Markdown
Collaborator Author

danmar commented Apr 24, 2026

There seems to be some kind of redirect, I didn't have to make any changes locally.

+1 thanks!
Do you see if the syncing from upstream works fine?

as far as I know, I only use my remote "upstream" when I want to access release branches.

@chrchr-github
Copy link
Copy Markdown
Collaborator

There seems to be some kind of redirect, I didn't have to make any changes locally.

+1 thanks! Do you see if the syncing from upstream works fine?

Pulls work, haven't tried much else yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants