Initial package creation of Kitematic.
Description
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | JoshStrobl | T2112 Kitematic | |||
| Resolved | JoshStrobl | T2660 Kitematic |
Event Timeline
Couple of things:
- I'm not sure if system.utils is the right component for this. It seems more like something for programming.tools
- You don't need binutils as a builddep. It's part of system.base.
- Just use install -d <dirname> for directory creation. It automatically sets 00755 as the permissions. The same goes for /usr/share.
- You should be using 00644 and 00755 for files modes. The leading 0's are important.
- The symlink creation should be the following or else it won't work:
ln -s /usr/share/kitematic/Kitematic $installdir/usr/bin/kitematic
Don't worry about a warning during the build for that symlink, it can't find it because it isn't actually there until it is installed. That's all.
- Does this need Docker for building? There some sort of libraries Kitematic needs from Docker? If neither, that could be removed from builddeps.
- If nodejs is needed for building, it's gonna be needed for running too, so that needs to be a rundep.
Hey @JoshStrobl,
- Good point. I can investigate this.
- Build process uses electron-prebuilt to build the app, so I believe this is already contained in the final application.
I will let you guys know how it goes.
Hey guys, sorry for the delay. It has been a few super busy days.
So here is my findings:
- No it doesn't need Docker to be built.
- The app brings it's own nodejs runtime. It does not need a nodejs runtime to be preinstalled.
The updated patch:
Outside of the *really minor* issue that the description should be after the summary and before the builddeps, this patch looks good to me. Solid work!
Currently apply work on top of the patch. It's evident now that this wasn't build with solbuild. Please do so in the future.
Resolved in unstable as of your patch and a follow-up one: https://git.solus-project.com/packages/kitematic/commit/?id=2845647da23587fa90198029e6a6cdf530783a18