Hi Felix,

On 11/01/16 03:28 PM, Felix Fietkau wrote:
> This is close to what I had in mind in my earlier response. Yes, it's
> ugly, but it's a lot less ugly than taking your approach without dealing
> with the issues that I pointed out.

Since a bit of ugliness for improved user experience seems to be acceptable, how do you feel about the following proposal to deal with current brokenness of ALL and ALL_KMODS (specifically that once there is an existing .config that selecting/deselecting those options no longer does anything; that is they only work when starting with no .config):

Make ALL a configuration option created by metadata.pl that does select <package> for all packages (except kmods), and ALL_KMODS a configuration option create by metadata.pl that does select <kmod-package> for all kmods and is default y if ALL.

This generated config could go under tmp and be sourced by Config-build.in in the current location of ALL and ALL_KMODS.

Even though it's kind of ugly, since you dislike the similar issues with profile selection as menu enough to accept some ugliness, I'm wondering if you could accept some ugliness to make ALL and ALL_KMODS work better too (as I'm not sure you'd even realized there was in issue before).

Regards,

Daniel
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to