-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Description
Hi. I just pushed meshroom (and alicevision and all the various dependencies) to Debian. So today if you're using Debian/sid you can apt install meshroom, and it will just work. When the downstream distros (Debian/stable, Ubuntu, etc) make their releases, they will get these packages too. This is infinitely easier than existing install methods, and should get more people to use this tool.
I had to patch the sources a bit to get things to work: https://salsa.debian.org/science-team/meshroom/-/tree/master/debian/patches?ref_type=heads
Ideally there would be no patches, and everything would build with appropriate flags. The patches:
-
https://salsa.debian.org/science-team/meshroom/-/blob/master/debian/patches/added-meshroom-executable.patch?ref_type=heads adds a
/usr/bin/meshrooomexecutable that the user can run. Can we add something like this to meshroom itself? Furthermore, the environment variables in that executable all point to standard locations on the system, so maybe they should be the default? -
https://salsa.debian.org/science-team/meshroom/-/blob/master/debian/patches/i-do-not-initsubmitters.patch?ref_type=heads removes the
initSubmitters()call. This uses something calledsimpleFarm, which isn't part of meshrooom. I don't know how this is supposed to work, but it should work without extra stuff like this patch -
The third patch removes the setup.py because I'm not using it at all to install meshroom. There's nothing to build, and I just move the files to the appropriate locations. Probably nothing to merge here.
There's a similar, but much longer list of patches for alicevision: https://salsa.debian.org/science-team/alicevision/-/tree/master/debian/patches?ref_type=heads I filed some bugs about that in the alicevision repo already. If some of those could be upstreamed, that would be great.
Thanks for meshroom!