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
F87545268: D41972.id127803.diff
Thu, Jul 4, 7:37 PM
F87513329: D41972.id127803.diff
Thu, Jul 4, 6:55 AM
Unknown Object (File)
Tue, Jun 25, 12:05 AM
Unknown Object (File)
May 28 2024, 6:48 PM
Unknown Object (File)
May 9 2024, 9:23 AM
Unknown Object (File)
May 9 2024, 9:23 AM
Unknown Object (File)
May 4 2024, 3:21 PM
Unknown Object (File)
May 4 2024, 2:44 AM
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
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
37–38

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
37–38

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.