-
-
Notifications
You must be signed in to change notification settings - Fork 15k
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
miniupnpc: 2.2.7 -> 2.2.8, again #326402
miniupnpc: 2.2.7 -> 2.2.8, again #326402
Changes from 17 commits
4b9186a
b0ee42d
579e16b
314f169
37087ef
2257420
2bf72c4
72446c1
58b9ac8
416aae7
edc7668
0abdd64
c9ba793
2e46cc8
77fbc57
0539f86
497259f
f96b9b9
5a149da
de9db75
b9f0aa8
7e44344
c5ff204
ca39d29
9511ce9
fcf6c25
defe4ae
38d971a
f5b70e5
5bc8c69
4d9b599
f67cebe
953b05c
956487c
b78f96a
8dd4be4
187349b
6875020
4a46bb9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,33 +1,32 @@ | ||
{ lib, mkDerivation, fetchFromGitHub | ||
, fetchpatch | ||
{ lib, stdenv, mkDerivation, fetchFromGitHub, fetchpatch2 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why is There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Ironically we probably need a |
||
, qmake, cmake, pkg-config, miniupnpc, bzip2 | ||
, speex, libmicrohttpd, libxml2, libxslt, sqlcipher, rapidjson, libXScrnSaver | ||
, qtbase, qtx11extras, qtmultimedia, libgnome-keyring | ||
}: | ||
|
||
mkDerivation rec { | ||
pname = "retroshare"; | ||
version = "0.6.6"; | ||
version = "0.6.7.2"; | ||
|
||
src = fetchFromGitHub { | ||
owner = "RetroShare"; | ||
repo = "RetroShare"; | ||
rev = "v${version}"; | ||
sha256 = "1hsymbhsfgycj39mdkrdp2hgq8irmvxa4a6jx2gg339m1fgf2xmh"; | ||
hash = "sha256-1A1YvOWIiWlP1JPUTg5Z/lxVGCBv4tCPf5sZdPogitU="; | ||
fetchSubmodules = true; | ||
}; | ||
|
||
patches = [ | ||
# The build normally tries to get git sub-modules during build | ||
# but we already have them checked out | ||
./no-submodules.patch | ||
./cpp-filesystem.patch | ||
|
||
# Fix gcc-13 build failure | ||
(fetchpatch { | ||
name = "gcc-13.patch"; | ||
url = "https://github.com/RetroShare/RetroShare/commit/e1934fd9b03cd52c556eb06d94fb5d68b649592e.patch"; | ||
hash = "sha256-oqxQAsD4fmkWAH2kSVmmed/q0LzTW/iqUU1SgYNdFyk="; | ||
# Support the miniupnpc-2.2.8 API change | ||
(fetchpatch2 { | ||
url = "https://github.com/RetroShare/libretroshare/commit/f1b89c4f87d77714571b4135c301bf0429096a20.patch?full_index=1"; | ||
hash = "sha256-UiZMsUFaOZTLj/dx1rLr5bTR1CQ6nt2+IygQdvwJqwc="; | ||
stripLen = 1; | ||
extraPrefix = "libretroshare/"; | ||
}) | ||
]; | ||
|
||
|
@@ -48,6 +47,15 @@ mkDerivation rec { | |
"RS_EXTRA_VERSION=" | ||
]; | ||
|
||
postPatch = '' | ||
# Build libsam3 as C, not C++. No, I have no idea why it tries to | ||
# do that, either. | ||
substituteInPlace libretroshare/src/libretroshare.pro \ | ||
--replace-fail \ | ||
"LIBSAM3_MAKE_PARAMS =" \ | ||
"LIBSAM3_MAKE_PARAMS = CC=$CC AR=$AR" | ||
''; | ||
|
||
postInstall = '' | ||
# BT DHT bootstrap | ||
cp libbitdht/src/bitdht/bdboot.txt $out/share/retroshare | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A bit weird to me how these external patches don't allow us to not have also a local patch... However this shouldn't block this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The previous anti‐vendoring patch was actually not fully functional in terms of specifying the header locations. The upstream PRs that fix miniupnpc 2.2.8 also forced the vendoring even harder. So since it’ll need dealing with anyway on the next version, I just fixed the anti‐vendoring stuff and reduced its size in the process.