split icon indicating copy to clipboard operation
split copied to clipboard

Something is wrong on split

Open DatL4g opened this issue 2 years ago • 3 comments

I currently switched from https://github.com/marketplace/actions/simple-split and something is off.

I'm using another action to sign a list of files. This will produce a string of the full path to the signed files, separated by a :.

The signing output string looks like this: /home/runner/work/Burning-Series/Burning-Series/app/release/app-release-signed.aab:/home/runner/work/Burning-Series/Burning-Series/app/release/app-release-signed.apk

My split config looks like this:

- name: Split signed files
  uses: JungWinter/[email protected]
  id: split
  with:
    msg: ${{ steps.sign_app.outputs.signedFiles }}
    separator: ':'
    maxsplit: 2

When I try to upload the files by their split path it can't find the first file.

So it tries to resolve /github/workspace/app/release/app-release-signed.aab but can't find it

However it works with the second file

You can take a look at the job here: https://github.com/DATL4G/Burning-Series/actions/runs/4398432795/jobs/7702280605

DatL4g avatar Mar 12 '23 17:03 DatL4g

Thank you for letting me know about the issue. It seems that there was an unexpected change in the workspace path mapping after the sign_app step. I'm guessing the split actions is docker-based, so it was affected. (refs. https://github.com/actions/runner/issues/763, https://github.com/actions/runner/issues/1009, https://github.com/actions/runner/issues/2058)

image

Upon checking the job log, I also found that /home/runner/work/Burning-Series/Burning-Series/app/release/app-release-signed.aab:/home/runner/work/Burning-Series/Burning-Series/app/release/app-release-signed.apk was passed into the split action, but it unintentionally became /github/workspace/app/release/app-release-signed.aab during the process.

To help with debugging, I created a new version of the [email protected], which you can try using by adding uses: JungWinter/[email protected] to your workflow. However, I'm not sure if this version can be used without marketplace publishing. If you encounter any issues, please let me know.

winterjung avatar Mar 13 '23 02:03 winterjung

I am facing the exact same issue. This /home/runner/work/nerd-clocksas/nerd-clocksas/app/build/outputs/app-release-signed.aab:/home/runner/work/nerd-clocksas/nerd-clocksas/app/build/outputs/app-release-signed.apk is being split into these /github/workspace/app/build/outputs/app-release-signed.aab /home/runner/work/nerd-clocksas/nerd-clocksas/app/build/outputs/app-release-signed.apk

@winterjung It does not work with uses: JungWinter/[email protected] nor does it throw an error.

supersu-man avatar Oct 09 '23 12:10 supersu-man

@supersu-man sorry for late replying. Could you try with uses: winterjung/[email protected] and share the debugging log? I renamed account name recently.

winterjung avatar Oct 12 '23 01:10 winterjung