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

a0b_armv7m_systick_clock 0.1.0 #1259

Conversation

godunko
Copy link
Contributor

@godunko godunko commented Oct 22, 2024

Created via alr publish with alr 2.0.2+9b80158

@godunko
Copy link
Contributor Author

godunko commented Oct 22, 2024

@mosteo failure looks strange, while legitimate. I suppose there was an issue with CI for #1253 where a0b_time was introduced. It should fails on Windows with the same error.

Do you have some recommendation to "fix" failure of the DOS line termination style check? I can just suppress it for a-reatim.adb, but same style check reported for many other files, so may be I need to configure Git repository instead?

@mosteo
Copy link
Member

mosteo commented Oct 22, 2024

Given that the failure pops up in specification files, maybe the problem is in the project here that's including the others. So I'd start by checking that the particular check is indeed not enabled. I've taken an initial look but I couldn't locate the a0b_common.gpr that's supplying the switches. Though, presuming this is shared by all projects, it's puzzling that the problem pops up only here. As you say, it might be related to the repo configuration, but I'm also presuming is the same for all your other repos...

@godunko
Copy link
Contributor Author

godunko commented Oct 22, 2024

a0b_common.gpr is in the a0b_base crate, see https://github.com/godunko/a0b-base/blob/main/gnat/a0b_common.gpr

Sorry for the noise, it is side effect of the -gnatg switch for a-reatim.adb file, compiler checks all files with-ed by it.

@godunko godunko marked this pull request as ready for review October 22, 2024 09:43
@mosteo mosteo merged commit 40e7b68 into alire-project:stable-1.3.0 Oct 22, 2024
18 checks passed
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.

2 participants