-
Notifications
You must be signed in to change notification settings - Fork 168
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
Make texlive's path to be unrelated to CPU architectures #482
Make texlive's path to be unrelated to CPU architectures #482
Conversation
- Remove (hard-coded) ARCH in PATH
@eitsupi The symbolic link is created before any |
might not be required, because |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for taking a look at this.
However, there may be no need to update these Dockerfiles at this stage as these can only be built on linux/arm64 linux/amd64.
I also wonder if adding the symbolic link to a location already added to the PATH would eliminate the need to update the PATH.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately, the build test failed.
And, suggestion to avoid linter complaining.
Make linter happy Co-authored-by: eitsupi <50911393+eitsupi@users.noreply.github.com>
@eitsupi Thanks for reviewing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working for this!
Fixes #478