Page MenuHomeFreeBSD

net/shadowsocks-libev: Add USES=compiler:c++11-lang and merge upstream patch (commit 662b6db3f3c395b552fa16f7fb22558aa38fe6aa) to fix build on GCC architectures.Tested on powerpc64 and amd64.PR: 237562
ClosedPublic

Authored by pkubaj on Apr 28 2019, 2:12 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 21, 12:30 AM
Unknown Object (File)
Mar 7 2025, 11:06 PM
Unknown Object (File)
Feb 27 2025, 6:31 PM
Unknown Object (File)
Feb 27 2025, 5:55 PM
Unknown Object (File)
Feb 21 2025, 10:18 PM
Unknown Object (File)
Feb 6 2025, 12:20 PM
Unknown Object (File)
Jan 31 2025, 4:28 PM
Unknown Object (File)
Jan 2 2025, 5:19 AM
Subscribers
None

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 23955
Build 22866: arc lint + arc unit

Event Timeline

linimon retitled this revision from Add USES=compiler:c++11-lang and merge upstream patch (commit 662b6db3f3c395b552fa16f7fb22558aa38fe6aa) to fix build on GCC architectures. Tested on powerpc64 and amd64. PR: 237562 to net/shadowsocks-libev: Add USES=compiler:c++11-lang and merge upstream patch (commit 662b6db3f3c395b552fa16f7fb22558aa38fe6aa) to fix build on GCC architectures.Tested on powerpc64 and amd64.PR: 237562.Apr 28 2019, 2:29 AM

I would try to add the upstream commit as patch, and call it patch-git_662b6d -- that way it is clear that it comes from upstream and can probably go in the next upgrade.

Also I would keep the whole header of the commit in there.

Merge patches to one file.

If this is a verbatim upstream patch, you could simply add a patch block:

PATCH_SITES=    https://github.com/${GH_ACCOUNT}/${GH_PROJECT}/commit/
PATCHFILES=     662b6db3f3c395b552fa16f7fb22558aa38fe6aa.patch:-p1 \

and run make makesum again.

Replace our local patch with patch from Github.

This revision is now accepted and ready to land.May 1 2019, 6:58 AM
This revision was automatically updated to reflect the committed changes.