Skip to content

Commit

Permalink
correct Feature/add actions (#7)
Browse files Browse the repository at this point in the history
* add first draft

* fix ubuntu-windows.yml

* fix ubuntu-windows.yml 2

* improve assets names

* fix typo

* try to fix release

* try to fix release 2

* try to fix release 2

* fix zip

* try to fix zip

* try to reduce centos packages
  • Loading branch information
sgatto authored Apr 7, 2022
1 parent 42f1b65 commit f94b2b0
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 10 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/centos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,15 +28,15 @@ jobs:
echo "RELEASE_CREATED=$release_created" >> $GITHUB_ENV
- name: Get release
if: ${{ env.RELEASE_CREATED == true }}
if: ${{ env.RELEASE_CREATED == 'true' }}
id: get_release
uses:
bruceadams/[email protected]

- name: install requirements
run: |
yum install -y epel-release
yum install -y git redhat-lsb-core make wget centos-release-scl scl-utils rpm-build
yum install -y git redhat-lsb-core centos-release-scl scl-utils
yum install -y cmake3 devtoolset-9
- name: Checkout sirius
Expand Down Expand Up @@ -81,10 +81,10 @@ jobs:
path: ${{ steps.sirius-variables.outputs.install-path }}

- name: Publish sirius install
if: ${{ env.RELEASE_CREATED == true }}
if: ${{ env.RELEASE_CREATED == 'true' }}
uses: actions/[email protected]
with:
upload_url: ${{ steps.get_release.outputs.upload_url }}
asset_path: ${{ steps.sirius-variables.outputs.install-path }}
asset_path: ${{ steps.sirius-variables.outputs.archive-path }}
asset_name: ${{ steps.sirius-variables.outputs.archive-name }}
asset_content_type: application/zip
10 changes: 4 additions & 6 deletions .github/workflows/ubuntu-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ jobs:
echo "RELEASE_CREATED=$release_created" >> $GITHUB_ENV
- name: Get release
if: ${{ env.RELEASE_CREATED == true }}
if: ${{ env.RELEASE_CREATED == 'true' }}
id: get_release
uses:
bruceadams/[email protected]
Expand All @@ -55,7 +55,7 @@ jobs:
ARCHIVE_PATH="${SIRIUS_BASE_PATH}/${ARCHIVE_NAME}"
echo "::set-output name=install-dir::$SIRIUS_INSTALL_DIR"
echo "::set-output name=install-path::$SIRIUS_INSTALL_PATH"
echo "::set-output name=sirius-archive-name::$ARCHIVE_NAME"
echo "::set-output name=archive-name::$ARCHIVE_NAME"
echo "::set-output name=archive-path::$ARCHIVE_PATH"
- name: Configure Sirius
Expand All @@ -72,11 +72,9 @@ jobs:
cmake --build build --config Release --target install -j14
- name: Zip Install Sirius
if : ${{ inputs.zip-sirius == 'true' }}
shell: bash
run: |
ARCHIVE_NAME="${{ steps.sirius-variables.outputs.archive-name }}"
zip -r $ARCHIVE_NAME "${{ steps.sirius-variables.outputs.install-dir }}"
zip -r "${{ steps.sirius-variables.outputs.archive-name }}" "${{ steps.sirius-variables.outputs.install-dir }}"
- name: Upload sirius install
uses: actions/upload-artifact@v2
Expand All @@ -85,7 +83,7 @@ jobs:
path: ${{ steps.sirius-variables.outputs.install-path }}

- name: Publish sirius install
if: ${{ env.RELEASE_CREATED == true }}
if: ${{ env.RELEASE_CREATED == 'true' }}
uses: actions/[email protected]
with:
upload_url: ${{ steps.get_release.outputs.upload_url }}
Expand Down

0 comments on commit f94b2b0

Please sign in to comment.