Skip to content

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

Open
pheest wants to merge 4 commits into
areaDetector:masterfrom
pheest:master
Open

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

Conversation

@pheest

@pheest pheest commented Jun 16, 2022

Copy link
Copy Markdown

No description provided.

@MarkRivers

MarkRivers commented Jun 16, 2022

Copy link
Copy Markdown
Member

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

pheest commented Jun 16, 2022

Copy link
Copy Markdown
Author

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

ericonr commented Dec 17, 2025

Copy link
Copy Markdown
Member

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

Were your ADCore and ADSupport changes merged?

@pheest

pheest commented Dec 31, 2025

Copy link
Copy Markdown
Author

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 thread urlApp/src/Makefile
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

ericonr commented Mar 17, 2026

Copy link
Copy Markdown
Member

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.

@pheest

pheest commented Jul 4, 2026

Copy link
Copy Markdown
Author

Hi, sorry for a long delay in responding.
I have been on prolonged holiday.

Yes, it would be possible to test the driver without hardware. This is a good idea.
There are some image files stored with the project, but note that the driver also supports use of streaming video; this would be more difficult to test in an automated way.
I had been testing the driver manually with a range of cameras that I located through http://www.insecam.org/

I don't know if the ADURL module is being CI-built for MinGW at present. This could indeed be done with GitHub Actions (rather that the GitLab server I was using at the time).

Most of what I changed related to deficiencies of the MinGW tool chain that may have been since fixed.
areaDetector/ADCore#481 was required, but notably did not fix the 'multiple definition of _Unwind_Resume on Linux' issue. 'back and forth changes' in ADSupport relate to this issue.

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