Opened 5 years ago
Closed 4 years ago
#60457 closed defect (fixed)
wxMaxima: destroot failure due to wrong case of wxmaxima.app
Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | MSoegtropIMC |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.6.2 |
Keywords: | Cc: | maehne (Torsten Maehne) | |
Port: | wxMaxima |
Description
wxMaxima fails to destroot on our build server:
Error: Failed to destroot wxMaxima: error copying "/opt/local/var/macports/build/_opt_bblocal_var_buildworker_ports_build_ports_math_wxMaxima/wxMaxima/work/wxmaxima-20.04.0/build/src/wxMaxima.app": no such file or directory
The message is correct: there is no wxMaxima.app; there is only wxmaxima.app. Case matters. Macs can use case-sensitive filesystems, and our build workers do.
One fix could be:
--- Portfile.orig +++ Portfile @@ -60,5 +60,5 @@ destroot { xinstall -m 755 -d ${destroot}${applications_dir} - file copy ${worksrcpath}/build/src/wxMaxima.app ${destroot}${applications_dir} + file copy ${worksrcpath}/build/src/wxmaxima.app ${destroot}${applications_dir}/wxMaxima.app }
Change History (5)
comment:1 Changed 5 years ago by MSoegtropIMC
comment:2 Changed 4 years ago by MSoegtropIMC
I am discussing with the wxMaxima developers if the name is wxmaxima or wxMaxima. The cmake files use fairly consistently the all lower case form, so one can argue that the correct fix would be to use that lower case name. But I will see what they answer. See: https://github.com/wxMaxima-developers/wxmaxima/issues/1220
comment:3 Changed 4 years ago by ryandesign (Ryan Carsten Schmidt)
As you like. I was trying to suggest a patch that would allow the port to install the files with the names you had originally specified, regardless of the case sensitivity of the filesystem.
If you want to change the names of the installed files, even just changing the case, that would mean increasing the port's revision as well. Or, to avoid an unnecessary rebuild, you could combine the rename with a version update, when one is available.
comment:4 Changed 4 years ago by maehne (Torsten Maehne)
Cc: | maehne added |
---|
comment:5 Changed 4 years ago by Michael Soegtrop <7895506+MSoegtropIMC@…>
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Thanks for reproting! I will try your suggested fix today.