Skip to content
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

pyside2@5.15.11_py310: rebottle for linux #512

Closed
wants to merge 1 commit into from

Conversation

ipatch
Copy link
Collaborator

@ipatch ipatch commented May 12, 2024

brew style freecad/freecad/[NAME_OF_FORMULA_FILE] 

output from running above command should output something similiar to the below

1 file inspected, no offenses detected
  • Have you ensured your commit passed audit checks, ie.
brew audit freecad/freecad/[NAME_OF_FORMULA_FILE] --online --new-formula

Not all PRs require passing these checks ie. adding [no ci] in the commit message will prevent the CI from running but PRs that change formula files generally should run through the CI checks that way new bottles are built and uploaded to the repository thus not having to build all formula from source but rather installing from a bottle (significantly faster 🐰 ... 🐢)

For more information about this template file learn more

@ipatch ipatch self-assigned this May 12, 2024
@ipatch
Copy link
Collaborator Author

ipatch commented May 13, 2024

there appears to be some limitations with tmate when connecting a github provider runner.

mxschmitt/action-tmate#184

mxschmitt/action-tmate#99

going to attempt to setup alternative to tmate, that hopefully 🫰 does not impose the limitations that tmate does.

https://github.com/lhotari/action-upterm

@ipatch ipatch force-pushed the pyside2-rebottle.for.linux branch 2 times, most recently from b0e5214 to d67150b Compare May 13, 2024 16:57
@ipatch
Copy link
Collaborator Author

ipatch commented May 15, 2024

in an attempt to not spam this repo with minor changes for debugging (figuring out why) the linux runner is not working with this repo. i updated my personal tap to troubleshoot the linux runner issues / errors. i will be working within that tap to hopefully resolve this issue with this repo. 🫰

ipatch/homebrew-us-05#7

@ipatch
Copy link
Collaborator Author

ipatch commented May 15, 2024

@ipatch
Copy link
Collaborator Author

ipatch commented May 15, 2024

https://github.com/FreeCAD/homebrew-freecad/actions/runs/9066820259/job/24912733114?pr=512#step:17:50

==> Running Formulae#run!
Error: Failed to determine dependencies for 'freecad/freecad/pyside2@5.15.11_py310'.
Error: Failed to determine dependencies for 'freecad/freecad/pyside2@5.15.11_py310'.
Error: Process completed with exit code 1.

@ipatch
Copy link
Collaborator Author

ipatch commented May 20, 2024

am able to reproduce the same errors i'm getting from this repo with my homebrew-us-05 repo copying the swig v4.1.1 formula file in this repo into my personal homebrew tap.

https://github.com/ipatch/homebrew-us-05/actions/runs/9162004441/job/25188066985

==> Testing ipatch/homebrew-us-05 9419013 (Merge 9a61d477846d7e5f0ff8927ed9fc172fe24a08da into 4d3ac87fe07578cf09717e35e077172f4286e548):

==> Running FormulaeDetect#detect_formulae!
==> git -C /home/linuxbrew/.linuxbrew/Homebrew/Library/Taps/ipatch/homebrew-us-05 fetch origin +refs/heads/dev
    url               https://github.com/ipatch/homebrew-us-05/pull/8/checks
    tap origin/dev 4d3ac87 (hello_world_c: add 1.0 bottle.)
    HEAD              9419013 (Merge 9a61d477846d7e5f0ff8927ed9fc172fe24a08da into 4d3ac87fe07578cf09717e35e077172f4286e548)
    diff_start_sha1   4d3ac87fe07578cf09717e35e077172f4286e548
    diff_end_sha1     941901392b3d3876c90ac4757e59ac8c17caadaa

    testing_formulae  ipatch/us-05/swig@4.1.1
    added_formulae    ipatch/us-05/swig@4.1.1
    modified_formulae (none)
    deleted_formulae  (none)
    formulae_to_fetch (none)

==> Running Formulae#run!
Error: Failed to determine dependencies for 'ipatch/us-05/swig@4.1.1'.
Error: Failed to determine dependencies for 'ipatch/us-05/swig@4.1.1'.
Error: Process completed with exit code 1.

@ipatch
Copy link
Collaborator Author

ipatch commented May 21, 2024

it appears the below env var is causing build issue with the github provided runner on linux

env:
HOMEBREW_NO_INSTALL_FROM_API: 1

 env:
          HOMEBREW_NO_INSTALL_FROM_API: 1

@ipatch ipatch force-pushed the pyside2-rebottle.for.linux branch from d67150b to dfedd63 Compare May 21, 2024 15:47
@ipatch
Copy link
Collaborator Author

ipatch commented May 21, 2024

got the below error / warnings but didn't seem to effect the run from building and uploading bottle artifacts 🤞

==> brew linkage --test freecad/freecad/pyside2@5.15.11_py310
==> brew linkage --cached --test --strict freecad/freecad/pyside2@5.15.11_py310
==> FAILED
Full linkage --cached --test --strict freecad/freecad/pyside2@5.15.11_py310 output
  Indirect dependencies with linkage:
    llvm
Truncated linkage --cached --test --strict freecad/freecad/pyside2@5.15.11_py310 output
  Warning: Indirect dependencies with linkage:
    llvm
  
==> brew linkage --cached freecad/freecad/pyside2@5.15.11_py310
==> brew install --only-dependencies --include-test freecad/freecad/pyside2@5.15.11_py310
==> brew test --verbose freecad/freecad/pyside2@5.15.11_py310

interestingly enough there was a discussion started recently in the homebrew discussions about indirect linkage when building bottle artifacts.

https://github.com/orgs/Homebrew/discussions/5368

@ipatch ipatch force-pushed the pyside2-rebottle.for.linux branch from dfedd63 to fa32ab1 Compare May 21, 2024 17:23
@ipatch
Copy link
Collaborator Author

ipatch commented May 21, 2024

add a revision line to the formula in attempt to resolve the below error,

With the provided path, there will be 2 files uploaded
Artifact name is valid!
Root directory input is valid!
Error: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run

the above mentioned error could possibly be related to the below,

actions/upload-artifact#478

@ipatch ipatch force-pushed the pyside2-rebottle.for.linux branch from fa32ab1 to 79f44b4 Compare May 21, 2024 19:44
@ipatch ipatch added the pr-pull label May 21, 2024
@github-actions github-actions bot closed this in 4ba862f May 21, 2024
@ipatch ipatch deleted the pyside2-rebottle.for.linux branch May 22, 2024 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant