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

Add Makefile to uniformly manage the compilation process #601

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

touero
Copy link
Contributor

@touero touero commented Nov 11, 2024

@touero
Copy link
Contributor Author

touero commented Nov 11, 2024

Associated with issue #600

@touero
Copy link
Contributor Author

touero commented Nov 15, 2024

Supplement mobile stealth.min.js and execute.sh and some questions to ask:

  • Can the suffix of execute_macos.sh be changed to mac64, it will be execute_mac64.sh
  • Can the suffix of package_macos.sh be changed to mac64, it will be package_mac64.sh

I see a lot of script files and can't modify them directly so here's a question for you.

If modified, the parameters in the makefile can be reduced.

@NaiboWang
Copy link
Owner

Supplement mobile stealth.min.js and execute.sh and some questions to ask:

  • Can the suffix of execute_macos.sh be changed to mac64, it will be execute_mac64.sh
  • Can the suffix of package_macos.sh be changed to mac64, it will be package_mac64.sh

I see a lot of script files and can't modify them directly so here's a question for you.

If modified, the parameters in the makefile can be reduced.

Yes, you can freely change the name of these scripts.

I will process all your pull requests after I finish my Ph.D. thesis next month, thank you again for your contribution!

@touero
Copy link
Contributor Author

touero commented Nov 20, 2024

  • Rename execute_macos.sh and package_macos.sh.
  • Add phony target in Makefile.

All is done. I often check the issues of this project and it can be of some help to other people's problems. I think I should apply to you for permission to label the issue. I don't know if this is possible.

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