Skip to content

Fix CI (follow-up of #244)#256

Merged
GitMensch merged 1 commit intoOCamlPro:gitside-gnucobol-3.xfrom
ddeclerck:fix_ci
Dec 3, 2025
Merged

Fix CI (follow-up of #244)#256
GitMensch merged 1 commit intoOCamlPro:gitside-gnucobol-3.xfrom
ddeclerck:fix_ci

Conversation

@ddeclerck
Copy link
Collaborator

@ddeclerck ddeclerck commented Nov 20, 2025

Follow-up of #244.

Opening a new PR, since the target branch was renamed.

However, I couldn't get the MinGW CI to work...

I'm using this opportunity to check whether the custom definition of HOME is still needed for the IBM CI.

@GitMensch
Copy link
Collaborator

just to note: you can change the target of a PR to another branch as well :-)

To not do things twice: what did you set BDB_LIBS to and did you already inspected which library files are installed?
Previously BDB was possibly built with a different toolchain (64bit) than the GnuCOBOL one (32 bit), but I think that should be fine with the last adjustments (maybe check twice, just to be sure).

And: yes, HOME can be dropped now for IBM, which is nice.

@ddeclerck
Copy link
Collaborator Author

just to note: you can change the target of a PR to another branch as well :-)

Oh, this sounds new.
The actual reason for the PR being closed is that I forced-pushed here after a rebase while there were no new commits...

To not do things twice: what did you set BDB_LIBS to and did you already inspected which library files are installed?

What it is currently set seems correct, there is indeed a file /mingw/lib/libdb-6.0.dll.a. But I tried without BDB_LIBS (as it worked before), which didn't help. Comparing with an older working version, I couldn't find why this changed.

And: yes, HOME can be dropped now for IBM, which is nice.

Yup, finally.

@GitMensch
Copy link
Collaborator

GitMensch commented Nov 20, 2025

I think we can use the linking feature of GCC, allowing it to link to dll's directly; not sure if the check run by configure passes that to gcc directly (but the logs should show that).

Just found out another broken issue: MSVC dependency artifacts are empty since a while (that did work before as well).

... CI saves time during development (or at least before a release), but it definitely needs some time on its own.

@ddeclerck
Copy link
Collaborator Author

Okay I think I finally found what was wrong.
BDB was compiled in 64-bit mode... Because configure "found" a cc command in /mingw64/bin (don't know where this one comes from...). So I just added CC=gcc to BDB's configure.

@GitMensch
Copy link
Collaborator

I guess you have to manually delete the BDB cache.

But the CC for BDB's configure should possibly either point to the full gcc path we want to use or we should export PATH directly before the configure step to have the mingw/bin coming first (I'd tend to the second option).

@GitMensch GitMensch merged commit 1fc514e into OCamlPro:gitside-gnucobol-3.x Dec 3, 2025
23 of 24 checks passed
@GitMensch
Copy link
Collaborator

Nice work! On the the open PRs amnd WIPs :-)

@ddeclerck ddeclerck deleted the fix_ci branch December 4, 2025 14:14
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