-
Notifications
You must be signed in to change notification settings - Fork 21
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-2.0: add new URL for SAS 9.4 M8 #634
base: master-2.0
Are you sure you want to change the base?
Conversation
docker-bits/6_sas.Dockerfile
Outdated
@@ -11,22 +11,39 @@ RUN wget -q ${QUARTO_URL} -O /tmp/quarto-${QUARTO_VERSION}-linux-amd64.tar.gz && | |||
chmod +x /tmp/quarto-${QUARTO_VERSION} && \ | |||
ln -s /tmp/quarto-${QUARTO_VERSION}/bin/quarto /usr/bin/quarto | |||
|
|||
RUN groupadd -g 1002 sasstaff && \ | |||
RUN groupadd -g 1337 supergroup && \ |
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.
This is reverting a PR I made. Is this actually necessary?
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.
It's just an old branch, I opened the branch before you made those changes, so this branch still had the old stuff. I've made the changes to match the work you did. Thank you for pointing that out. I should pay more attention when making pull requests for old branches.
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.
There was another user/permissions error:
Line 33:
chown -R sas:sasstaff /usr/local/SASHome --> chown -R jovyan:sasstaff /usr/local/SASHome
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.
This update is to remove the SAS image's dependency on sas4c, which is the older SAS image using SAS 9.4 M7.
I have uploaded SAS 9.4 M8 to an Azure storage container and moved the relevant logic out of sas4c into the AAW SAS image.
Todo:
Ticket: https://jirab.statcan.ca/browse/BTIS-304