way-displays icon indicating copy to clipboard operation
way-displays copied to clipboard

Added package to nixpks

Open simoneruffini opened this issue 2 years ago • 9 comments

Hi! as of today way-displays can be installed via nix package manager (Linux distros using nix and NixOS). The package is available from the unstable branch of nixpks. Probably will be merged in the next release as stable.

I am the current maintainer for that package, I will update it when new versions will be available. But if any of you wants to do it the necessary changes to update the package are realy simple.

For the future we can think to ad a CI/CD process so that the package can be directly downloaded from GitHub instead of building it on the host platform.

simoneruffini avatar Jun 16 '22 12:06 simoneruffini

That's fantastic, thanks @simoneruffini

Everything looks smooth except for the /etc patch

Can we change the code so that you don't have to apply that patch? Is there perhaps an environment variable that we could use?

alex-courtis avatar Jun 20 '22 07:06 alex-courtis

For the future we can think to ad a CI/CD process so that the package can be directly downloaded from GitHub instead of building it on the host platform.

That would be ideal. It only needs to be done for Linux as FreeBSD already have their own build process.

Time is limited...

alex-courtis avatar Jun 20 '22 07:06 alex-courtis

Everything looks smooth except for the /etc patch

Can we change the code so that you don't have to apply that patch? Is there perhaps an environment variable that we could use?

I will ask some NixOS gurus on how to do that, because I don't know if it is a good idea to have a env variable that let's you change the default position of the configuration files (the one you find in ../etc). I will check how this is get accomplished.

simoneruffini avatar Jun 24 '22 12:06 simoneruffini

I will check how this is get accomplished.

Perhaps a #define with an /etc default?

alex-courtis avatar Jun 25 '22 00:06 alex-courtis

So this means that a patch will be necessary anyway? Just in a more precise spot?

simoneruffini avatar Jun 25 '22 13:06 simoneruffini

I am aiming for no patches. Patches are fragile and need maintenance. We've removed all patches from the BSD and other distro's builds and I'd like to do the same here.

~~We could perhaps simply use PREFIX_ETC in the makeFlags as a #define and add another search in cfg.cpp e.g.~~

	if (!found)
		found = resolve_paths(cfg, PREFIX_ETC"/etc", "");

~~However, other package builds may use DESTDIR and PREFIX* to target their staging area, so using those might not be a good idea.~~

~~Best solution would be a specific makeFlag -> #define for this case, maybe FHS_ETC or ROOT_ETC, defaulting to /etc.~~

PR speaks louder than words: #48

alex-courtis avatar Jun 26 '22 01:06 alex-courtis

Merged. Do you need a release @simoneruffini ?

alex-courtis avatar Jun 29 '22 23:06 alex-courtis

I closed this issue because the package is available but maybe we can keep it open until I check the other one

simoneruffini avatar Jun 29 '22 23:06 simoneruffini

1.6.0 release PR for nixpkgs https://github.com/NixOS/nixpkgs/pull/182041

alex-courtis avatar Jul 19 '22 04:07 alex-courtis