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

Submodules #4234

Merged
merged 10 commits into from
Jan 5, 2025
Merged

Submodules #4234

merged 10 commits into from
Jan 5, 2025

Conversation

UjuiUjuMandan
Copy link
Contributor

@UjuiUjuMandan UjuiUjuMandan commented Jan 4, 2025

Continuing:

I want to keep your manual actions to as less as possible, and moving all external repos to submodules.

Since the updaing of AndroidLibXrayLite was automated, you only need to check the submodule to corresponding latest tag, the same tag string as xray-core, inside current repo.

@2dust
Copy link
Owner

2dust commented Jan 5, 2025

Thanks for your work
Please remove support for AndroidLibV2rayLite.
Because the AndroidLibV2rayLite project is already difficult to maintain.
It will take a lot of modifications to run properly when released.

@UjuiUjuMandan
Copy link
Contributor Author

UjuiUjuMandan commented Jan 5, 2025

Emmm... Is the reverting really necessary? I'm not supporting it or so, it's only for compiling libtun2socks.

Only the build script of libtun2socks is used.

@2dust
Copy link
Owner

2dust commented Jan 5, 2025

Remove AndroidLibV2rayLite,
You can copy the script for compiling libtun2socks to AndroidLibXrayLite

@UjuiUjuMandan
Copy link
Contributor Author

OK, that should work.

@UjuiUjuMandan
Copy link
Contributor Author

I'll open another pull request in AndroidLibXrayLite of migrating libtun2socks.

@UjuiUjuMandan

This comment was marked as outdated.

@UjuiUjuMandan

This comment was marked as resolved.

@UjuiUjuMandan
Copy link
Contributor Author

@2dust 2dust merged commit dcb003f into 2dust:master Jan 5, 2025
1 check passed
@UjuiUjuMandan UjuiUjuMandan deleted the submodules branch January 5, 2025 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants