Skip to content
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.

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

torcs: anonscm.debian.org has moved #58978

Merged
merged 1 commit into from Apr 4, 2019
Merged

Conversation

eri451
Copy link
Contributor

@eri451 eri451 commented Apr 4, 2019

Motivation for this change

the patch links where somewhat broken
my guess it is due to the move to a gitlab at salsa.debian.org

Things done

changed patch urls to non access required download links

  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

the patch links where somewhat broken
my guess it is due to the move to a gitlab at salsa.debian.org
I used the non access required download link generated by gitlab
@worldofpeace
Copy link
Contributor

You need to pin these patches to a commit since master is mutable.

So look at https://salsa.debian.org/games-team/torcs/tree/master/debian/patches
and pick the latest commit for the patch in the url

https://salsa.debian.org/games-team/torcs/raw/fb0711c171b38c4648dc7c048249ec20f79eb8e2/debian/patches/gcc6-isnan.patch

eri451 added a commit to eri451/nixpkgs that referenced this pull request Apr 11, 2019
the patch links returned a HTTP error 401 as
anonscm.debian.org moved to a gitlab at salsa.debian.org
I used the non access required download link generated by the new instance
As @worldofpeace suggested in NixOS#58978 I used the last commit involving the files.
@eri451 eri451 mentioned this pull request Apr 11, 2019
10 tasks
c0bw3b pushed a commit to c0bw3b/nixpkgs that referenced this pull request Apr 24, 2019
* torcs: anonscm.debian.org has moved

the patch links returned a HTTP error 401 as
anonscm.debian.org moved to a gitlab at salsa.debian.org
I used the non access required download link generated by the new instance
As @worldofpeace suggested in NixOS#58978 I used the last commit involving the files.

* torcs: add patch on OpenALMusicPlayer.cpp

Unbreak builds with GCC7

(cherry picked from commit 99b9166)
c0bw3b pushed a commit that referenced this pull request Apr 24, 2019
* torcs: anonscm.debian.org has moved

the patch links returned a HTTP error 401 as
anonscm.debian.org moved to a gitlab at salsa.debian.org
I used the non access required download link generated by the new instance
As @worldofpeace suggested in #58978 I used the last commit involving the files.

* torcs: add patch on OpenALMusicPlayer.cpp

Unbreak builds with GCC7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants