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

Ubuntu CI fails PRs. #2438

Closed
clanmills opened this issue Dec 21, 2022 · 7 comments · Fixed by #2439
Closed

Ubuntu CI fails PRs. #2438

clanmills opened this issue Dec 21, 2022 · 7 comments · Fixed by #2439
Labels

Comments

@clanmills
Copy link
Collaborator

Describe the bug

CI fails PRs. For example: #2437

To Reproduce

Submit a PR with trivial non-code change (for example change a document). CI will fail the Ubuntu builds

Expected behavior

CI Should succeed.

Desktop (please complete the following information):

This issue is on the CI, so no Desktop is involved.

Additional context

Example:
https://github.com/Exiv2/exiv2/actions/runs/3730282452/workflow

The error message is

-- Set runtime path of "/home/runner/work/exiv2/exiv2/build-base_linux/install/bin/geotag" to "$ORIGIN/../lib"
[94]install [error opening dir]

I believe this message is generated by the following code in the workflow file:

      - name: Install
        run: |
          cd build
          cmake --install . 
          tree install  <--- error generated here
@clanmills clanmills added the bug label Dec 21, 2022
@clanmills
Copy link
Collaborator Author

I believe this is related to change d2083b3 which removed -DCMAKE_INSTALL_PREFIX=install from the cmake command and results in

tree install
correctly failing to find the directory install.

IIRC, tree was used help debug build issues and can be safely removed from the workflow script.

Having satisfied my curiosity about the cause, I'm not going to submit a PR/fix.

@clanmills
Copy link
Collaborator Author

I've made the change and submitted it as fix_2438_ci_ubuntu. The git monster has eaten it. No idea what has happened to it. When I submit the PR request, it says, "main and fix_2438_ci_ubuntu are identical". Ssshshsh.

diff --git a/.github/workflows/on_PR_linux_matrix.yml b/.github/workflows/on_PR_linux_matrix.yml
index 95fbc9f8..3f36c704 100644
--- a/.github/workflows/on_PR_linux_matrix.yml
+++ b/.github/workflows/on_PR_linux_matrix.yml
@@ -50,7 +50,6 @@ jobs:
         run: |
           cd build
           cmake --install . 
-          tree install
 
       - name: Test
         run: |

@clanmills
Copy link
Collaborator Author

I appear to have insufficient privileges:

refusing to allow an OAuth App to create or update workflow `.github/workflows/on_PR_linux_matrix.yml` without `workflow` scope
559 rmills@rmillsmm-local:/Volumes/rmills/gnu/github/exiv2/main $ git push origin fix_2438_ci_ubuntu
Enumerating objects: 27, done.
Counting objects: 100% (27/27), done.
Delta compression using up to 8 threads
Compressing objects: 100% (5/5), done.
Writing objects: 100% (5/5), 419 bytes | 0 bytes/s, done.
Total 5 (delta 4), reused 0 (delta 0)
remote: Resolving deltas: 100% (4/4), completed with 4 local objects.
remote: This repository moved. Please use the new location:
remote:   https://github.com/Exiv2/exiv2.git
To https://github.com/exiv2/exiv2
 ! [remote rejected]   fix_2438_ci_ubuntu -> fix_2438_ci_ubuntu (refusing to allow an OAuth App to create or update workflow `.github/workflows/on_PR_linux_matrix.yml` without `workflow` scope)
error: failed to push some refs to 'https://github.com/exiv2/exiv2'
560 rmills@rmillsmm-local:/Volumes/rmills/gnu/github/exiv2/main $ 

@piponazo
Copy link
Collaborator

Hi Robin! Happy to see you here.

I think you have correctly identified where the error more or less lies, although I am not sure why this started to happen suddenly. I´ll take a look at it.

By the way, by looking to your last console snippet, it looks like you might be pointing to an old remote:

remote: This repository moved. Please use the new location:
remote:   https://github.com/Exiv2/exiv2.git

Do you have something like this when you run git remote -v?

git remote -v
origin  git@github.com:Exiv2/exiv2.git (fetch)
origin  git@github.com:Exiv2/exiv2.git (push)

@clanmills
Copy link
Collaborator Author

@piponazo Merry Christmas and Best Wishes for 2023.

I edited .git/config without success. Maybe I should pull down a fresh copy of the code from github.com:Exiv2/exiv2.git.

Using the command line isn't easy, as github.com does not accept passwords.

@postscript-dev
Copy link
Collaborator

@clanmills
Merry Christmas Robin.

I appear to have insufficient privileges:

I don't know if this is helpful but when I had OAuth problems in the past, it was because my token was expired or improperly configured.

@clanmills
Copy link
Collaborator Author

@piponazo Thanks. I see you've dealt with this (and many other matters) in #2439. That's wonderful. I can go back to my music and leave you to deal with the code.

So I wish you and everybody a Peaceful and Prosperous 2023.

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 a pull request may close this issue.

3 participants