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

improve maketool logics #829

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

improve maketool logics #829

wants to merge 2 commits into from

Conversation

yslan
Copy link
Contributor

@yslan yslan commented Apr 25, 2024

  • Standard help mode "-h" / "--help"

  • Go through whole list before error exit
    To test this, set MAXNEL=15000000000 and only n2to3 will success

  • Add "core" mode for essential tools ./maketool core
    Currently, LIST_CORE="genmap genbox n2to3 reatore2 nekmerge"

  • Add seperate clean mode via "./maketool clean LIST"
    For example, this only clean genmap and gencon
    ./maketool clean genmap gencon

  • Fix CI TOOL test (need verify)
    Change CI test to "core" mode, not "all"

  • TODO: need to check whether NekDoc needs update

- Standard help mode "-h" / "--help"

- go through whole list before error exit
  To test this, set MAXNEL=15000000000 and only n2to3 will success

- Add "core" mode for essential tools
  ./maketool core
  Change CI test for "core" mode, not "all"

- Add seperate clean mode via "./maketool clean LIST"
  For example, this only clean genmap and gencon
  ./maketool clean genmap gencon
@yslan
Copy link
Contributor Author

yslan commented Apr 25, 2024

Only tested with bash. Need a zsh user to try

@stgeke stgeke marked this pull request as draft May 8, 2024 06:40
@stgeke
Copy link
Contributor

stgeke commented May 8, 2024

Converting to draft. Please change it when you think the PR is ready for review.

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.

None yet

2 participants