Closed Simranpal closed 4 years ago
@diegoakechi We are not following the archive.tar
conventions here. I'm quite happy with this naming and methods. Otherwise, we would need change the module to have something like sapcar
instead of extract
and pass the options from the state. We can do that aswell.
What do you think @Simranpal @diegoakechi ?
https://docs.saltstack.com/en/latest/ref/modules/all/salt.modules.archive.html
https://docs.saltstack.com/en/latest/ref/states/all/salt.states.archive.html
PD: The state naming is the same
@arbulu89 It was more a suggestion from my side, but I am ok to proceed as it is right now.
Add the salt functions to extract the SAP sar package.
I created new file names as
saputils
to add future sap related functions. The option to extract rar,exe SAP installation package can also be added here, what do you think?And I kept the
output_dir
is optional, and if its not provided, then use current directory as sar_file to extract.TO-Do: Add unit tests and make any further improvements
Related to: https://github.com/SUSE/sapnwbootstrap-formula/pull/41 https://github.com/SUSE/shaptools/pull/49