Skip to content

Build ADURL with ci-scripts on Ubuntu using out GitLab server.#9

Open
pheest wants to merge 4 commits intoareaDetector:masterfrom
pheest:master
Open

Build ADURL with ci-scripts on Ubuntu using out GitLab server.#9
pheest wants to merge 4 commits intoareaDetector:masterfrom
pheest:master

Conversation

@pheest
Copy link
Copy Markdown

@pheest pheest commented Jun 16, 2022

No description provided.

@MarkRivers
Copy link
Copy Markdown
Member

MarkRivers commented Jun 16, 2022

This PR contains 3 large .log files. Do these belong in the repository, or should they be added to .gitignore?

@MarkRivers
Copy link
Copy Markdown
Member

There is still one more log file.

@pheest
Copy link
Copy Markdown
Author

pheest commented Jun 16, 2022

Oops, sorry about that. I've deleted the files from the request.

NB, the build scripts reference my (much more significant) pull requests at ADCore and ADSupport.
This should of course be changed after these have been processed.

@ericonr
Copy link
Copy Markdown
Member

ericonr commented Dec 17, 2025

This runs gitlab CI. Is there any way to send the results back into this repository?

Were your ADCore and ADSupport changes merged?

@pheest
Copy link
Copy Markdown
Author

pheest commented Dec 31, 2025

Hi, this is so long ago, I am struggling to remember the details.

The point of the project was to CI build areaDetector modules with MinGW on an Ubuntu runner.
This was not easy.

No, there is no way now to upload the results, but I did get it working.

My related pull request(s) are:
areaDetector/ADSupport#39 (remains open). That is more significant than this one.
areaDetector/ADCore#481 (was closed Jan 1 2024).

I am inclined to say this PR should be deleted, but preferably the ADSupport PR should be merged.

Comment on lines +30 to +34
ifeq (mingw, $(findstring mingw, $(T_A)))
# multiple definition of `_Unwind_Resume on Linux. I don't know how to fix it.
USR_LDFLAGS += -Wl,-allow-multiple-definition
endif

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documenting for posterity, this was likely fixed by areaDetector/ADCore#481 ?

@ericonr
Copy link
Copy Markdown
Member

ericonr commented Mar 17, 2026

I could close this, or we could move it to a GitHub Actions setup. What do you think? It's a driver that can be built easily and tested without any hardware.

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