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

main: add args to override paths to allow non-root usage #24

Merged
merged 1 commit into from
May 7, 2024

Conversation

jiegec
Copy link
Member

@jiegec jiegec commented May 7, 2024

Example usage:

acbs-build -gw bash -o $PWD/../logs -l $PWD/../cache -z $PWD/../temp -b $PWD

Copy link

what-the-diff bot commented May 7, 2024

PR Summary

  • Introduction of New Command Line Arguments for acbs-build
    The acbs-build script has now been integrated with new command line arguments. These allow for more control of the script, helping streamline the process of application control and specification of build options.

  • Conditional Handling of Command Line Arguments in acbs/main.py
    The main Py file now contains conditional statements to cater for the newly introduced command line arguments. The inclusion of these equips the codebase with the ability to respond to varying inputs, enhancing flexibility, and usability.

  • Update to Initialization Method in acbs/main.py
    The initialization method within the primary Py file has been fine tuned. It is now designed to adequately handle situations where the acbs_tree_dir hasn't been provided through the command line argument. This improvement guarantees a level of automation, making the script more fault-tolerant and reliable.

  • Error Management for Missing forest.conf File
    This update includes a safeguard for unfortunate scenarios where the forest.conf file cannot be located. The enhanced error handling ensures the platform maintains its operational continuity, minimizing disruptions due to unforeseen issues with configuration files.

@MingcongBai MingcongBai merged commit 37accc2 into production May 7, 2024
1 check passed
@MingcongBai
Copy link
Member

Agreement reached with @liushuyu to test this in production.

@KexyBiscuit KexyBiscuit deleted the add-args-override-dir branch July 21, 2024 03:27
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