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

Debug missing stable27 updates #16

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

nickvergessen
Copy link
Contributor

No description provided.

@nickvergessen nickvergessen force-pushed the debug-missing-stable27-updates branch from 443ac5e to c83881e Compare October 4, 2023 09:11
@nickvergessen nickvergessen force-pushed the debug-missing-stable27-updates branch from c83881e to d22f967 Compare October 4, 2023 09:18
@nickvergessen nickvergessen merged commit fa6ebb3 into master Oct 4, 2023
@nickvergessen nickvergessen deleted the debug-missing-stable27-updates branch October 4, 2023 09:22
@nickvergessen
Copy link
Contributor Author

Adding this debug output fixed stable27 not being updated since a month:
3dddb78

But even now it claimed that only build.sh was modified:
https://github.com/nextcloud-deps/ocp/actions/runs/6404034874/job/17383766850#step:4:45

but then the actual git commit listed the added files successfully:
https://github.com/nextcloud-deps/ocp/actions/runs/6404034874/job/17383766850#step:4:53

The linked file should be there since 7th September when this PR was merged:
nextcloud/server#40286

@nickvergessen
Copy link
Contributor Author

It also did not work for the tags:
https://github.com/nextcloud-deps/ocp/commits/v27.1.1/OCP
the tags are missing the respective file as well.

@nickvergessen
Copy link
Contributor Author

Uff, so the problem is git diff does not list untracked files and no file was modified, only new files added.
Can be fixed by always git add OCP and then using git diff --cached, so will go for that.

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.

1 participant