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

Jump to bottom

libupnp: 1.12.0 -> 1.14.0 #93048

Merged
merged 1 commit into from Oct 7, 2020
Merged

libupnp: 1.12.0 -> 1.14.0 #93048

merged 1 commit into from Oct 7, 2020

Conversation

ardumont
Copy link
Contributor

@ardumont ardumont commented Jul 13, 2020 •
edited

Trying to package gerbera (upnp media server) which requires at least that
dependency version in its build recipe.

  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS,
    • macOS
    • other: debian with nix
  • 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 nixpkgs-review pr 93048 [1]
  • Tested execution of all binary files (usually in ./result/bin/)
  • Impact on package closure size: + 1352 (33560928 -> 33562280)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

[1] 3 packages broke with this, last one (obs-ndi is in the same state as master, needs a license to build):

$ nix-shell -p nixpkgs-review --run "nixpkgs-review pr 93048"
...
6 package marked as broken and skipped:
libsForQt5.vlc libsForQt512.vlc libsForQt514.vlc retroshare retroshare06 ring-daemon

4 package failed to build:
amule amuleDaemon gmrender-resurrect obs-ndi

Those 3 are fixed with the following PR:

This was referenced Jul 16, 2020
Merged
Merged
@risicle
Copy link
Contributor

risicle commented Sep 5, 2020

libupnp itself builds happily on macos 10.14 (all reverse dependencies on macos are marked broken).

@risicle
Copy link
Contributor

risicle commented Sep 6, 2020 •
edited

Actually I've noticed libupnp 1.14.0 is now out, and it fixes https://nvd.nist.gov/vuln/detail/CVE-2020-13848 so it'd be useful to rebase & update this PR.

@ardumont
Copy link
Contributor Author

ardumont commented Sep 6, 2020

Actually I've noticed libupnp 1.14.0 is now out, and it fixes https://nvd.nist.gov/vuln/detail/CVE-2020-13848 so it'd be useful to
rebase & update this PR.

thanks for the heads up, i'll check and adapt.

@ardumont ardumont changed the title libupnp: 1.12.0 -> 1.12.1 libupnp: 1.12.0 -> 1.14.0 Sep 6, 2020
timokau
timokau requested changes Oct 6, 2020
View reviewed changes
Copy link
Member

@timokau timokau left a comment

Choose a reason for hiding this comment

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

nix-review tells me that this breaks three packages that built previously: amule, amuleDaemon and gmrender-resurrect.

@ardumont
Copy link
Contributor Author

ardumont commented Oct 6, 2020

nix-review tells me that this breaks three packages that built previously: amule, amuleDaemon and gmrender-resurrect.

Do you have some links about the proper way to deal with those package breaks?
Thanks for your time.

Cheers,

@timokau
Copy link
Member

timokau commented Oct 6, 2020

If you have some time to spare, you could have a look at the build log and see if there is anything obvious that needs fixing. It could also be helpful to check out the upstream bug ticker, or if there are any new releases available that might be compatible.

If you do not have the spare time right now, you can ping the maintainers of the broken packages here and give them ~a week to respond. They can then fix the issue, or if they don't respond you can mark the packages as broken.

@ardumont
Copy link
Contributor Author

ardumont commented Oct 6, 2020

I concur:

$ nix-shell -p nixpkgs-review --run "nixpkgs-review pr 93048"
...
6 package marked as broken and skipped:
libsForQt5.vlc libsForQt512.vlc libsForQt514.vlc retroshare retroshare06 ring-daemon

4 package failed to build:
amule amuleDaemon gmrender-resurrect obs-ndi

I checked against a more recent master build which render the same results (so i updated that diff).

@ardumont ardumont mentioned this pull request Oct 6, 2020
Merged
10 tasks
@ardumont
Copy link
Contributor Author

ardumont commented Oct 6, 2020

#99893 fixes amule and amuleDaemon by bumping the version

@ardumont
Copy link
Contributor Author

ardumont commented Oct 6, 2020 •
edited

obs-ndi

That's because it requires to validate a license for the build, it does not relate to this.
(It's in the same state as the master branch)

@ardumont
Copy link
Contributor Author

ardumont commented Oct 6, 2020

gmrender-resurrect

I looked into that one as well and a PR is already opened to fix
the build against libupnp 1.14 upstream [1]

[1] hzeller/gmrender-resurrect#214

@ardumont ardumont mentioned this pull request Oct 6, 2020
Merged
10 tasks
@ardumont
Copy link
Contributor Author

ardumont commented Oct 6, 2020

gmrender-resurrect
I looked into that one as well and a PR is already opened to fix
the build against libupnp 1.14 upstream [1]

[1] hzeller/gmrender-resurrect#214

And there we go, #99899 should fix that last one.

ardumont added a commit to ardumont/nixpkgs that referenced this pull request Oct 6, 2020
@ardumont
amule: 2.3.2 -> unstable-20201006
1f7cf96 
There has been no new release since 2.3.2 for some years now (around 2016) [1]

The most recent master branch for the amule project includes a commit [2] which
fixes compilation breakage with the next libupnp version [3].

So this bumps the amule version to the latest commit of that repository.

[1] amule-project/amule#219

[2] amule-project/amule@8784480

[3] NixOS#93048
@ardumont
Copy link
Contributor Author

ardumont commented Oct 7, 2020

Updated the summary with the other PR references.

timokau
timokau approved these changes Oct 7, 2020
View reviewed changes
Copy link
Member

@timokau timokau left a comment •
edited

Choose a reason for hiding this comment

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

nixpkgs-review is happy now that #99893 and #99899 are in. Thanks!

@timokau timokau merged commit c7f33a5 into NixOS:master Oct 7, 2020
@ardumont
Copy link
Contributor Author

ardumont commented Oct 7, 2020

\o/ Awesome \m/

Cheers,

@ardumont ardumont deleted the bump-libupnp branch October 7, 2020 08:14
@FliegendeWurst FliegendeWurst mentioned this pull request Nov 27, 2021
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@timokau timokau timokau approved these changes

Assignees
No one assigned
Labels
10.rebuild-darwin: 1-10 10.rebuild-linux: 11-100
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ardumont @risicle @timokau