i can never remember if it's FEATURES=makecheck or FEATURES=maketest, either way, there's no reason to have the 'make' in there since the src_test() function could run w/out make with some packages
ebuild.sh still refers to FEATURES=maketest and really, that's the one that matters the most in terms of backwards support, i dont think it's that critical since maketest has been a semi-unstable feature thus far ... just rename everything to 'test' and remove all traces of 'maketest' and call it a day
Out already.