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

batman-adv: update packages to version 2024.2 #1071

Merged
merged 3 commits into from
Jun 24, 2024

Conversation

ecsv
Copy link
Contributor

@ecsv ecsv commented Jun 23, 2024

Maintainer: @simonwunderlich
Compile tested: x86_64
Run tested: x86_64

Description:

batman-adv

  • support latest kernels (4.19 - 6.10)
  • coding style cleanups and refactoring

ecsv added 2 commits June 22, 2024 21:33
* (no changes)

Signed-off-by: Sven Eckelmann <[email protected]>
* support latest kernels (4.19 - 6.10)
* coding style cleanups and refactoring

Signed-off-by: Sven Eckelmann <[email protected]>
* (no changes)

Signed-off-by: Sven Eckelmann <[email protected]>
@simonwunderlich
Copy link
Contributor

Thank you! Just for the record, the build server had some issues with downloading on the first two builds as open-mesh.org had some temporary problems, but they are fixed.

@simonwunderlich simonwunderlich merged commit 6d376d6 into openwrt:master Jun 24, 2024
9 of 11 checks passed
@BKPepe
Copy link
Member

BKPepe commented Jun 24, 2024

I wonder, why you wouldnt retrigger the CI/CD before merging this?

@ecsv
Copy link
Contributor Author

ecsv commented Jun 24, 2024

I can't. I can create a status badge but not trigger a rebuild. I would have to do again a useless commit and force-push again the "no-change".

@BKPepe
Copy link
Member

BKPepe commented Jun 24, 2024

Even though it might not seem like a good way to do it, it is still better to retrigger it like this so we are sure that those temporary issues are gone and that the CI/CD passes. Also, there is another option to ask someone with the commit permission to retrigger it. I did it after the merging, but it seems that CI/CD still can not download it.

Right now, it looks like we do have CI/CD, and it complains about one issue, but we are ignoring it as we can. It is not always a great idea, though.

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