While correcting quoting of linuxtv-dvb-apps I found a case where repoman (portage-2.1.4_rc3) failed to detect an issue. Part of ebuild: emake bindir=/usr/bin datadir=/usr/share libdir=/usr/$(get_libdir) \ prefix=/usr DESTDIR=${D} INSTDIR=${T} install || die "install failed" No warning is displayed.
The EbuildQuote check currently doesn't track state across line continuations. We want to try not to generate false positives for variable assignments that don't need quoting, like this line by itself does not require quoting if it is just variable assignments rather than command arguments: DESTDIR=${D} INSTDIR=${T}
with line continuations, i dont think we'd get any more false positives by collapsing the lines so we'd take: foo \ moo \ boo collapse it down to: foo moo boo and then feed it to the checker
This has been fixed for some time now: http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=a1578c654f26cab07309bc9cbddd3c95c0c205b5
