Skip to content

gpsd: unbreak the build for guiSupport=false#182173

Merged
bjornfor merged 2 commits intomasterfrom
unknown repository
Jul 28, 2022
Merged

gpsd: unbreak the build for guiSupport=false#182173
bjornfor merged 2 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jul 20, 2022

Description of changes

The rm $out/bin/xgps* command fails because that file is not there in the first place. Let's change it to rm -f $out/bin/xgps* so we can build with guiSupport=false.

Things done
  • Built on platform(s)
    • x86_64-linux
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

The `rm $out/bin/xgps*` command fails because that file is not there in the first place.  Let's change it to `rm -f $out/bin/xgps*` so we can build with `guiSupport=false`.
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Jul 20, 2022
@ghost ghost mentioned this pull request Jul 21, 2022
5 tasks
@ghost
Copy link
Copy Markdown
Author

ghost commented Jul 27, 2022

Ping @bjornfor @rasendubi

@vcunat
Copy link
Copy Markdown
Member

vcunat commented Jul 27, 2022

Do I understand that the file is never there? (assuming guiSupport == false) Why not delete that line then?

@ghost
Copy link
Copy Markdown
Author

ghost commented Jul 27, 2022

Do I understand that the file is never there? (assuming guiSupport == false)

I believe that is correct.

Why not delete that line then?

An abundance of caution. But you're right, leaving dead code there is worse. I've removed it in the latest commit.

@ofborg ofborg bot requested review from bjornfor and rasendubi July 27, 2022 19:13
@ofborg ofborg bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. and removed 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Jul 27, 2022
@bjornfor bjornfor merged commit 248f618 into NixOS:master Jul 28, 2022
@vcunat
Copy link
Copy Markdown
Member

vcunat commented Jul 28, 2022

It fails the same on 22.05.

@github-actions
Copy link
Copy Markdown
Contributor

Successfully created backport PR #183323 for release-22.05.

@ghost ghost deleted the pr/gpsd/guiSupportFalse branch July 28, 2022 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants