Skip to content

transmission_4: init at 4.0.3#241146

Merged
dywedir merged 4 commits into
NixOS:masterfrom
chvp:init/transmission_4
Jul 15, 2023
Merged

transmission_4: init at 4.0.3#241146
dywedir merged 4 commits into
NixOS:masterfrom
chvp:init/transmission_4

Conversation

@chvp

@chvp chvp commented Jul 2, 2023

Copy link
Copy Markdown
Member
Description of changes

https://github.com/transmission/transmission/releases/tag/4.0.0
https://github.com/transmission/transmission/releases/tag/4.0.1
https://github.com/transmission/transmission/releases/tag/4.0.2
https://github.com/transmission/transmission/releases/tag/4.0.3

Previous discussion (on why this isn't just an update) in #215316.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • 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/)
  • 23.11 Release Notes (or backporting 23.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

@chvp chvp force-pushed the init/transmission_4 branch from f7ac8ef to 4ea37dd Compare July 2, 2023 18:01
@ofborg ofborg Bot added the 8.has: package (new) This PR adds a new package label Jul 2, 2023
@ofborg ofborg Bot requested a review from astsmtl July 2, 2023 18:21
@ofborg ofborg Bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Jul 2, 2023
@ofborg ofborg Bot requested a review from emilytrau July 2, 2023 18:33
@chvp chvp force-pushed the init/transmission_4 branch from 60c271f to c7de8d4 Compare July 3, 2023 07:26
@chvp chvp force-pushed the init/transmission_4 branch from c7de8d4 to c8db41c Compare July 3, 2023 08:09
@chvp

chvp commented Jul 3, 2023

Copy link
Copy Markdown
Member Author

Getting this to build on x86_64-darwin will not be easy. transmission 4 needs macOS >= 10.14.6. The current build failure is due to some constants missing from the CommonCrypto headers.

@chvp chvp mentioned this pull request Jul 3, 2023
13 tasks
Comment thread pkgs/applications/networking/p2p/libutp/3.4.nix Outdated
@chvp chvp force-pushed the init/transmission_4 branch from 1425fe2 to a58c858 Compare July 3, 2023 09:39
@chvp chvp self-assigned this Jul 3, 2023
@chvp chvp removed their assignment Jul 3, 2023
@chvp chvp added the 12.approvals: 2 This PR was reviewed and approved by two persons. label Jul 4, 2023
@chvp

chvp commented Jul 6, 2023

Copy link
Copy Markdown
Member Author

I've been using this PR instead of #215316 in my nixos-config without issues ever since creating it.

@nixos-discourse

Copy link
Copy Markdown

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-already-reviewed/2617/1017

@dywedir

dywedir commented Jul 15, 2023

Copy link
Copy Markdown
Member

Thanks!

@dywedir dywedir merged commit be2968a into NixOS:master Jul 15, 2023
@chvp chvp deleted the init/transmission_4 branch July 15, 2023 17:27
@linsui linsui mentioned this pull request Aug 31, 2023
12 tasks
@kirillrdy

Copy link
Copy Markdown
Member

@chvp may I ask why nixosTest wasn't carried over from transmission 3 ? seems to be still passing for me

@chvp

chvp commented Sep 23, 2023

Copy link
Copy Markdown
Member Author

Forgetting to do so, probably.

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

Labels

8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.approvals: 2 This PR was reviewed and approved by two persons.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants