Pascal Stumpf wrote: > On Sun, 15 Aug 2021 08:54:42 +0200, Stefan Hagen wrote: > > Stefan Hagen wrote: > > > Pascal Stumpf wrote: > > > > On Fri, 13 Aug 2021 18:36:59 -0700, Daniel Menche wrote: > > > > > Built it and played a few campaign missions; it works great. > > > > > > > > > > 'sequences.wz', the archive containing the campaign cutscene videos, > > > > > is still missing. As it is not required for play and is nearly a > > > > > gigabyte, I think it should be put in a separate package. > > > > > > > > I think I'd prefer just putting some installation instructions into a > > > > README. > > > > > > I have another idea. We could make it flavors. > > > > > > With the diff below, the port can create these files: > > > > > > 340M warzone2100-4.1.3.tgz > > > 501M warzone2100-4.1.3-videolow.tgz > > > 885M warzone2100-4.1.3-video.tgz > > > 1.2G warzone2100-4.1.3-videohigh.tgz > > > > > > I think three flavors for the same thing are a bit much. Should we pick > > > one or two? > > > > > > I also fixed a few issues portcheck was complaining about: > > > * added databases/sqlite3 to LIB_DEPENDS > > > * moved /share/icons to /share/warzon2100/icons > > > * added COMPILER line > > > > > > And one from my own check-list: > > > * added NO_TESTS=Yes > > > > Again, with corrected distinfo and SUPDISTFILES. > > This is nothing short of insane. Nobody will ever install any of these > flavours. They will just collect dust on the mirrors (and eat up >1GB > of of disk space/bandwidth on every sync).
What is insane? Flavoring all three versions? Yes, but then you haven't read the full mail. One flavor with the standard or low version would still be insane? The whole idea came up because I don't think instructing a user to download a non-checksummed file from somewhere and put it into /usr/local/..., which requires root and sticks around after pkg_delete is a good idea either. But hey, let's be constructive here... I'm currently looking if sequence.ws can also be found if we put it in $HOME/.local/share/warzone2100-4.1/. There is an ifdef for MACOS in main.cpp, which shows how it works. I'd like to have this as fallback for those that don't want to manually touch /usr/local. Then I'm fine with the readme approach. Best Regards, Stefan
