Adjust for FreeCAD v1 release#6
Conversation
There was a problem hiding this comment.
Thanks for opening this pull request! Be sure to follow the pull request template!
|
I am a bot, here are the test results for this PR:
|
|
I am a bot, here are the test results for this PR:
|
e4c72d1 to
ed38bee
Compare
ed38bee to
724167f
Compare
|
I am a bot, here are the test results for this PR:
|
|
I am a bot, here are the test results for this PR:
|
|
I am a bot, here are the test results for this PR:
|
Roxedus
left a comment
There was a problem hiding this comment.
Changing the location we ingest a application from does also need a chance in jenkins-vars, as well as a changelog entry.
| echo " install freecad from appimage " && \ | ||
| freecad_version="1.0.0" && \ | ||
| cd /tmp && \ | ||
| curl -o /tmp/freecad.app -L https://github.com/FreeCAD/FreeCAD/releases/download/1.0.0/FreeCAD_1.0.0-conda-Linux-x86_64-py311.AppImage && \ |
There was a problem hiding this comment.
We are not going to accept a hard coded version number.
There was a problem hiding this comment.
replaced with environment variable
|
Question, maybe suggestion, but would using the "continous" version be accepted? Instead of using: Use: Though perhaps, the cleanest way is to use |
|
I am a bot, here are the test results for this PR:
|
|
Hi, Thanks for your efforts. Any chance to be available in the official docker image ? Still stucked in 0.20, and I have some projects started in 1.0... Thanks for all Ps : sorry for my English |
While in genereal I like the idea, I'd prefer to have a idemporent behaviour. |
|
Where do I adjust the environment variable in Jenkins? |
|
you dont |
|
The merge build is failing because I looked around the repository to see where this env var is set, but I couldn't find anything, except usages of it, which I guess means that the version is set through jenkins somehow and not through the files. So I guess unless the version env var is bumped this can't work. Also since we will now be using the appimage, maybe we should delete the following packages from the
|
|
There isn't anything you can do as a 3rd party to get Jenkins to build using your version logic, because we specifically prevent it for security reasons. This https://github.com/linuxserver/docker-freecad/blob/master/jenkins-vars.yml#L5-L6 needs changing, probably to something like https://github.com/linuxserver/docker-webcord/blob/master/jenkins-vars.yml#L5-L13 as this uses the GH release and the Dockerfiles themselves need something like https://github.com/linuxserver/docker-webcord/blob/master/Dockerfile#L19-L23 to fetch the latest version for scenarios where it's not passed into the build process. Finally the readme-vars will need a new changelog entry to document the change of install process. package_versions.txt is an output not an input so does not need to be touched. |
If you look at pr #7 then you will see. That was my attempt at a better approach to switch yogithub. Jenkins currently uses a apt command to see existing version available in the repository of the Linux distribution. I that pr I attempted to switch jenkins to use a github api lookup to get the latest version. I asked on discourse why the jenkins build fail for my pr. @Roxedus |
Have a look at pr #7 which is my attemot to switch to github releases. Currently jenkins is configured to look up latest version from the apt repo. I tried to switch to the github api for relates version lookup, but jenkins still run the old config files from the existing repo before the pr, even for pr builds. This is what @Roxedus explained to me discourse. I am however in hospital so can't take up that conversation further. But somehow, a pr will have to be accepted to get an updated jenkins config into the main branch. |
|
This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions. |
|
Not sure what I need to do to get this merged... |
|
I apologize I did not realize this image was out of maintenance. But after looking at this and #7 I will prioritize this tomorrow and update this PR with a test image you can put through it's paces does that sound ok? We have a bunch of internal process stuff you need to do to modify ingestion, most of the participation we get is adaptive not restructuring. |
|
@jaecktec @tinuva please test this image out min run command Land on https://host:3001 you can use nvidia with |
Tested the image by replacing the image line in my docker compose without the nvidia bits. I think this image sums up what I found.
I then tried the following: Then |
|
So you are seeing that because you are mounting in the old home directory, that is a good testing point, the menu and autostart file are the old launchers. From a clean context it just works. I will add logic to overwrite those files if they are not updated. I'll also make the symlink no harm in that. |
|
ok @tinuva I updated the image to have a binary link, it is the cleanest solution you are correct there. Please pull If there are things missing here that users need now is the time. The only thing I can say with certainty that will be fixed is the ugly cursor, that is going to be done in the base image though. I just want to know about stuff like this plugin does not work or , like what users normally plug into this, if they need a taskbar for window management? |
|
Please chime in on #8 if you have feedback |




Description:
Adjusted Dockerfile according to suggestions to support Freecad 1.0 with AppImage
Benefits of this PR and context:
Freecad 1.0
How Has This Been Tested?
I've build this on an M2 Macbook + X86 Linux machine and ran them
Source / References:
#4