Hi, > -----Original Message----- > From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org] > On Behalf Of Jonas Gorski > Sent: Samstag, 14. September 2019 11:54 > To: Paul Spooren <m...@aparcar.org> > Cc: Sergey Ryazanov <ryazanov....@gmail.com>; Tomasz Maciej Nowak > <tome...@o2.pl>; Roman Yeryomin <ro...@advem.lv>; Tim Harvey > <thar...@gateworks.com>; Luka Perkov <l...@openwrt.org>; Jason Wu > <jason.wu.m...@gmail.com>; Alexander Couzens <lyn...@fe80.eu>; John > Crispin <j...@phrozen.org>; OpenWrt Development List <openwrt- > de...@lists.openwrt.org>; Felix Fietkau <n...@nbd.name> > Subject: Re: [OpenWrt-Devel] [PATCH] treewide: add Generic subtarget if > missing > > On Fri, 23 Aug 2019 at 11:04, Paul Spooren <m...@aparcar.org> wrote: > > As in 853e4dd OpenWrt should follow a unified structure, where every > > device has a target/subtarget combination, if there is only one > > subtarget, call it "Generic". This introduces predictable filenames. > > If it's about (I assume generated) filenames, wouldn't it be easier to just > use > "Generic" for the subtarget part of the filename if there are no subtargets? > I'm not really a fan of unnecessary code fluff without any real function, > especially if it means additional, mainly empty files.
What you suggest is about what we have right now. This kind of creates a misleading situation where for some targets subtargets are present, while for others paths and image names are "fixed" in several places to include a "generic". The reason for Paul's patch was to get rid of the fixes at individual places (which was/is applied somewhat inconsistently) by just making all targets apply to the same logic (i.e. that there is at least one subtarget). So, the empty files are introduced to make the process of building and creating images afterwards simpler (to follow/understand). I was suffering from the same problem when I dealt with OpenWrt-derived firmware, where I suddenly encountered a target without subtargets and had to implement extra code to work around that. However, I also wondered whether one couldn't code around the necessity of the empty file, and just add the SUBTARGET/SUBTARGETS variables here... Best Adrian > > Regards > Jonas > > _______________________________________________ > openwrt-devel mailing list > openwrt-devel@lists.openwrt.org > https://lists.openwrt.org/mailman/listinfo/openwrt-devel
openpgp-digital-signature.asc
Description: PGP signature
_______________________________________________ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/mailman/listinfo/openwrt-devel
