setup.sh: install latexindent perl dependencies #2
Conversation
setup.sh
Outdated
| cpanm -n File::HomeDir | ||
| cpanm -n Params::ValidationCompiler | ||
| cpanm -n YAML::Tiny | ||
| cpanm -n Unicode::GCString |
There was a problem hiding this comment.
It seems that you can install all these packages from apk. Please do so to avoid unnecessary dependencies.
You should be able to find the package name from https://pkgs.alpinelinux.org/packages
There was a problem hiding this comment.
It seems some packages like perl-file-homedir are in testing repo, so I try to add http://dl-cdn.alpinelinux.org/alpine/edge/testing in /etc/apk/repositories
Line 26 in 15aa124
setup.sh
Outdated
| apk --no-cache add curl wget make | ||
| cd /usr/local/bin && \ | ||
| curl -L https://cpanmin.us/ -o cpanm && \ | ||
| chmod +x cpanm |
There was a problem hiding this comment.
We don't want to introduce unnecessary dependencies. Please find if the Perl dependencies can be installed from apk directly.
setup.sh
Outdated
| perl-log-dispatch \ | ||
| perl-namespace-autoclean \ | ||
| perl-specio \ | ||
| perl-unicode-linebreak |
There was a problem hiding this comment.
Please move these to the "install system package" in the above.
|
Thanks @2014CAIS01. |
May be fix #1 but now I run these command in https://github.com/xu-cheng/texlive-action manually, but I think the setup should add in
setup.shAppendices: https://latexindentpl.readthedocs.io/en/latest/appendices.html#lst-alpine-install