Skip to content

pdfpc: 4.4.1 -> 4.5.0 #108307

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

Merged
merged 1 commit into from
Jan 3, 2021
Merged

pdfpc: 4.4.1 -> 4.5.0 #108307

merged 1 commit into from
Jan 3, 2021

Conversation

wentasah
Copy link
Contributor

@wentasah wentasah commented Jan 3, 2021

Motivation for this change

New upstream version.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • 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 nixpkgs-review --run "nixpkgs-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)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

Sorry, something went wrong.

Verified

This commit was signed with the committer’s verified signature. The key has expired.
lukekarrys Luke Karrys
Copy link
Member

@mdlayher mdlayher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

$ nixpkgs-review pr 108307
$ git -c fetch.prune=false fetch --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0 pull/108307/head:refs/nixpkgs-review/1
remote: Enumerating objects: 5, done.
remote: Counting objects: 100% (5/5), done.
remote: Total 7 (delta 5), reused 5 (delta 5), pack-reused 2
Unpacking objects: 100% (7/7), 660 bytes | 330.00 KiB/s, done.
From ssh://github.com/NixOS/nixpkgs
   2497a55f5f0..4d0e996b71d  master                -> refs/nixpkgs-review/0
   168f36153b1..9d0164d53d4  refs/pull/108307/head -> refs/nixpkgs-review/1
$ git worktree add /home/matt/.cache/nixpkgs-review/pr-108307/nixpkgs 4d0e996b71d48ac0155ffc0a4de38a4c4743debd
Preparing worktree (detached HEAD 4d0e996b71d)
HEAD is now at 4d0e996b71d Merge pull request #108303 from siraben/codec2-darwin
$ git merge --no-commit 9d0164d53d42417e19f9cea858801fef2cf01768
Automatic merge went well; stopped before committing as requested
$ nix build --no-link --keep-going --option build-use-sandbox relaxed -f /home/matt/.cache/nixpkgs-review/pr-108307/build.nix
[3 built, 266 copied (1103.2 MiB), 245.2 MiB DL]
https://github.com/NixOS/nixpkgs/pull/108307
1 package built:
pdfpc

$ nix-shell /home/matt/.cache/nixpkgs-review/pr-108307/shell.nix
these paths will be fetched (0.51 MiB download, 2.47 MiB unpacked):
  /nix/store/8fmzmylr9l60yir9qzixyksrfdshnan6-bash-interactive-4.4-p23-doc
  /nix/store/i539vammbyynzv6rh55aq5l79k79awcx-bash-interactive-4.4-p23-dev
  /nix/store/j9bq7mdvqrncsk9bprbc47gcwr422k94-bash-interactive-4.4-p23-man
  /nix/store/qd5vfykrf9cag5swl66rg5b10jlpzmwb-bash-interactive-4.4-p23-info
copying path '/nix/store/8fmzmylr9l60yir9qzixyksrfdshnan6-bash-interactive-4.4-p23-doc' from 'https://cache.nixos.org'...
copying path '/nix/store/i539vammbyynzv6rh55aq5l79k79awcx-bash-interactive-4.4-p23-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/qd5vfykrf9cag5swl66rg5b10jlpzmwb-bash-interactive-4.4-p23-info' from 'https://cache.nixos.org'...
copying path '/nix/store/j9bq7mdvqrncsk9bprbc47gcwr422k94-bash-interactive-4.4-p23-man' from 'https://cache.nixos.org'...

@pSub pSub merged commit 9c9c646 into NixOS:master Jan 3, 2021
@vbgl
Copy link
Contributor

vbgl commented Jan 27, 2021

Thanks for this update!
According to the changelog, the license of pdfpc has changed to gpl3Plus.

@wentasah
Copy link
Contributor Author

Thanks for noticing that. I'm sending a PR with an update.

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

4 participants