| 18 Mar 2023 |
Alyssa Ross | Yeah | 15:02:28 |
Alyssa Ross | While you're at it, might be good to flip the default of enableBloat, because it's pretty confusing to see it default to false in the package, but then have it overridden to true in the diffoscope callPackage | 15:03:09 |
raitobezarius | got it | 15:03:22 |
raitobezarius | Alyssa Ross: hm it's more complicated | 15:06:20 |
raitobezarius | diffoscopeMinimal = callPackage ../ { jdk = jdk8; } | 15:06:33 |
raitobezarius | diffoscope = diffoscopeMinimal.override { enableBloat = true; } | 15:06:41 |
raitobezarius | * diffoscope = diffoscopeMinimal.override { enableBloat = !stdenv.isDawrin; } | 15:06:45 |
raitobezarius | * diffoscope = diffoscopeMinimal.override { enableBloat = !stdenv.isDarwin; } | 15:06:48 |
raitobezarius | so would you say I move the stdenv.isDarwin to the derivation? | 15:06:55 |
Alyssa Ross | yes | 15:07:03 |
raitobezarius | okay | 15:07:13 |
Alyssa Ross | And have diffoscopeMinimal override diffoscope instead. | 15:07:16 |
raitobezarius | got it | 15:07:22 |
raitobezarius | is there a good way to test if a package is unfree also? | 15:08:36 |
raitobezarius | like I can write the filterUnfree function with examining pkg.meta.license(s).free parameter | 15:08:50 |
raitobezarius | just wondering if there was a better way | 15:08:57 |
Alyssa Ross | You can, but it's probably best not to. | 15:09:00 |