OK, I will adjust and update the .install file. Also what I will do is use directly your git source and maybe also additionally install the README to /etc/gtk-3.0 or so…
Please have a look:
Great. One small issue: it seems to me that
/usr/share/ is better for REAMDE file rather than
/etc/ is for configuration files. In my opinion
/usr/share/gtk3-classic/README.md would be better.
Second very important thing. If you are using my
gtk3-mushrooms repo in your PKGBUILD, you should use specified commit hash instead of the newest commit from
master branch. When I do update, I can break your PKGBUILD (sometimes new patches can be incompatible with old GTK version).
/usr/share/doc/gtk3-classic/README.md would be even better.
@oberon. Please update your
gtk3-classic. I moved patching code from
PKGBUILD to separated script
do-mushrooms.sh. You can run this script from your
Ok, thanks. Done
Word is spreading…
(See that post and the three that follow)
I think I’ll have to consider this for bspwm edition. I have no problems with csd, but I really like typeahead. And smaller packages…
I think this should probably be done with proper patch file rather than a shell script.
is where it’s at. If I have time I’ll put a PR in to https://github.com/TomaszGasior/gtk3-mushrooms
@TomaszGasior Just for ask … is possible divide the pkgs ? Create a Group gtk3-classic , in this way is possible install example only typehead function or other … or is a too work ?
I don’t think it is possible to separate functions that way. What you could have is three separate packages gtk3, gtk3-typeahead, gtk3-nocsd and gtk3-classic. But imo that is a bit too much.
What i mean is like this : same pkgbuild which build all pkgs in one time …
That should be possible.
This seems overcomplicated as the options are set by an environment variable. If you want a default for a spin or edition, set the variables in e.g.
I agree. No need for separate package where configuration is enough.
My intent was more general and for our user , not a big problem but an user can still use only one of this patch but in this way you force to install all set
Understandable. Though I’d hope users wouldn’t install this without knowing what it does.