Page MenuHomeFreeBSD

security/xray-core: Update to 1.8.4
ClosedPublic

Authored by jbo on Sep 25 2023, 11:27 AM.
Tags
None
Referenced Files
F134521239: D41972.diff
Sun, Nov 2, 9:03 PM
Unknown Object (File)
Sat, Nov 1, 11:00 PM
Unknown Object (File)
Wed, Oct 29, 3:39 AM
Unknown Object (File)
Sun, Oct 26, 2:12 AM
Unknown Object (File)
Sat, Oct 18, 12:28 PM
Unknown Object (File)
Mon, Oct 13, 9:38 PM
Unknown Object (File)
Sun, Oct 12, 5:27 PM
Unknown Object (File)
Sun, Oct 12, 5:14 PM
Subscribers

Details

Summary

Changelogs:

1.8.4: https://github.com/XTLS/Xray-core/releases/tag/v1.8.4
1.8.3: https://github.com/XTLS/Xray-core/releases/tag/v1.8.3
1.8.1: https://github.com/XTLS/Xray-core/releases/tag/v1.8.1
1.8.0: https://github.com/XTLS/Xray-core/releases/tag/v1.8.0

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 53711
Build 50602: arc lint + arc unit

Event Timeline

jbo requested review of this revision.Sep 25 2023, 11:27 AM

Add PR reference to commit message.

I asked the submitter about the removal of the rc.d script:

rc.d script is not from upstream. I created it as a part of 1.7.5 update. Script still exists and being installed with the port. It is templatized and located in files/xray.in

I accidentally added it to pkg-list in previous version while it is not required based on my understanding of template system. I clearly see rc.d templated scripts not to be part of pkg-list in other ports. I validated the port still installs/removes script properly without any errors.

security/xray-core/Makefile
40

Are these files really needed for the build or could we install them directly from DISTDIR in post-install?

jbo marked an inline comment as done.

Remove seemingly unnecessary copy of *.dat files during build.

Currently waiting on submitter feedback whether this change is okay.

security/xray-core/Makefile
40

Good point. I ran some test builds without this and they seem to succeed just fine.

Submitter responded that this is okay.

This revision is now accepted and ready to land.Sep 25 2023, 4:35 PM
This revision was automatically updated to reflect the committed changes.