Skip to content

ci: debug docker build with new actions #311

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.

Sign up for GitHub

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

Closed
wants to merge 2 commits into from
Closed

ci: debug docker build with new actions #311

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e999d26
ci: debug docker build with new actions
apaletta3 Jan 10, 2025
94e72f3
Update Dockerfile
apaletta3 Jan 10, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 24 additions & 19 deletions docker/development/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,15 @@

# General purpose development image (root user)

FROM openspacecollective/open-space-toolkit-base-development:${BASE_IMAGE_VERSION} as root-user

Check warning on line 7 in docker/development/Dockerfile

View workflow job for this annotation

GitHub Actions / Build Development Image / Build Development Image

The 'as' keyword should match the case of the 'from' keyword 

FromAsCasing: 'as' and 'FROM' keywords' casing do not match
More info: https://docs.docker.com/go/dockerfile/rule/from-as-casing/

LABEL maintainer="lucas@loftorbital.com"

# Test
ARG TARGETPLATFORM

RUN echo "TARGETPLATFORM: ${TARGETPLATFORM}"

# Dependencies

## Zip, jq, and git-lfs
Expand Down Expand Up @@ -89,25 +94,25 @@
&& cp -r *.a /usr/local/lib \
&& rm -rf /tmp/sofa

## SPICE Toolkit

ARG TARGETPLATFORM

RUN cd /tmp \
&& export PACKAGE_PLATFORM=$(if [ ${TARGETPLATFORM} = "linux/amd64" ]; then echo "PC_Linux_GCC_64bit"; elif [ ${TARGETPLATFORM} = "linux/arm64" ]; then echo "MacM1_OSX_clang_64bit"; else echo "Unknown platform" && exit 1; fi;) \
&& wget --quiet http://naif.jpl.nasa.gov/pub/naif/toolkit/C/${PACKAGE_PLATFORM}/packages/cspice.tar.Z \
&& tar -xf cspice.tar.Z \
&& cd cspice \
&& apt-get update \
&& apt-get install -y csh \
&& ./makeall.csh > /dev/null \
&& mkdir -p /usr/local/include/cspice \
&& cp -r include/* /usr/local/include/cspice/ \
&& cp -r lib/* /usr/local/lib/ \
&& ln -s /usr/local/lib/cspice.a /usr/local/lib/libcspice.a \
&& rm -rf /tmp/cspice \
&& apt-get remove -y csh \
&& rm -rf /var/lib/apt/lists/*
# ## SPICE Toolkit

# ARG TARGETPLATFORM

# RUN cd /tmp \
# && export PACKAGE_PLATFORM=$(if [ ${TARGETPLATFORM} = "linux/amd64" ]; then echo "PC_Linux_GCC_64bit"; elif [ ${TARGETPLATFORM} = "linux/arm64" ]; then echo "MacM1_OSX_clang_64bit"; else echo "Unknown platform" && exit 1; fi;) \
# && wget --quiet http://naif.jpl.nasa.gov/pub/naif/toolkit/C/${PACKAGE_PLATFORM}/packages/cspice.tar.Z \
# && tar -xf cspice.tar.Z \
# && cd cspice \
# && apt-get update \
# && apt-get install -y csh \
# && ./makeall.csh > /dev/null \
# && mkdir -p /usr/local/include/cspice \
# && cp -r include/* /usr/local/include/cspice/ \
# && cp -r lib/* /usr/local/lib/ \
# && ln -s /usr/local/lib/cspice.a /usr/local/lib/libcspice.a \
# && rm -rf /tmp/cspice \
# && apt-get remove -y csh \
# && rm -rf /var/lib/apt/lists/*

# GeographicLib

Expand Down Expand Up @@ -192,7 +197,7 @@

# Development image for humans (non-root user)

FROM root-user as non-root-user

Check warning on line 200 in docker/development/Dockerfile

View workflow job for this annotation

GitHub Actions / Build Development Image / Build Development Image

The 'as' keyword should match the case of the 'from' keyword 

FromAsCasing: 'as' and 'FROM' keywords' casing do not match
More info: https://docs.docker.com/go/dockerfile/rule/from-as-casing/

# Install dev utilities

Expand Down
Loading