Skip to content

Conversation

@kaniini
Copy link
Contributor

@kaniini kaniini commented Aug 21, 2023

Some packages used mv to move files into the subpkgdir after building them. Now that we have ${{targets.contextdir}}, this extra mv step is no longer needed: the go/build pipeline now follows the context dir instead.

Some packages used `mv` to move files into the subpkgdir after building
them.  Now that we have ${{targets.contextdir}}, this extra `mv` step
is no longer needed: the go/build pipeline now follows the context dir
instead.

Signed-off-by: Ariadne Conill <[email protected]>
@kaniini kaniini requested a review from a team as a code owner August 21, 2023 19:10
@github-actions
Copy link
Contributor

Package bank-vaults: Click to expand/collapse

Package bank-vaults:
Added: /usr/bin/template

Package gitlab-runner: Click to expand/collapse

Package gitlab-runner:
Added: /usr/bin/gitlab-runner-helper

Package guac: Click to expand/collapse

Package guac:
Added: /usr/bin/guaccsub
Added: /usr/bin/guacgql
Added: /usr/bin/guacingest
Added: /usr/bin/guacone

Package guaccsub: Click to expand/collapse

Package guaccsub:
Deleted: /usr/bin/guaccsub

Package bank-vaults-template: Click to expand/collapse

Package bank-vaults-template:
Deleted: /usr/bin/template

Package gitlab-runner-helper: Click to expand/collapse

Package gitlab-runner-helper:
Deleted: /usr/bin/gitlab-runner-helper

Package guacingest: Click to expand/collapse

Package guacingest:
Deleted: /usr/bin/guacingest

Package guacone: Click to expand/collapse

Package guacone:
Deleted: /usr/bin/guacone

Package guacgql: Click to expand/collapse

Package guacgql:
Deleted: /usr/bin/guacgql

@kaniini
Copy link
Contributor Author

kaniini commented Aug 21, 2023

These CI results are to be expected, as they depend on the new Melange changes

@rawlingsj rawlingsj added this pull request to the merge queue Aug 22, 2023
Merged via the queue into main with commit d3ed539 Aug 22, 2023
@rawlingsj rawlingsj deleted the chore/melange-go-build-changes branch August 22, 2023 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants