The handbook says binaries should not be added to the portage tree (introduced by bug https://bugs.gentoo.org/show_bug.cgi?id=331331 ) However it also says they can be added if you use "cvs add -kb myphoto.png". From the discussion in bug https://bugs.gentoo.org/show_bug.cgi?id=417117 it seems they should _NEVER_ be added. If so, this is a documentation bug and should be clarified.
It seems I didn't read closely enough, cause it says: "If you need to do this in another CVS tree" However I think this can potentially lead to misunderstanding, especially because the following cvs command is shown in nice color with the title "Adding binary files to CVS".
The text could be moved into it's own /green "note" box/ to stand out more. If you really want to pursue this, provide a patch.
Created attachment 313165 [details, diff] hb-guide-ebuild.xml.patch I just removed the example and notes about how to add binary files to (other) CVS cause this is not a general guide to CVS commands and can lead to confusion. I find this cleaner.
-1. The snippet about adding binary files is useful for the gentoo/ tree, but not the gentoo-x86/ tree.
Yes that is true. Other cvs trees (docs/ etc) can have binary files. That paragraph is clear enough that no binary files are allowed in gentoo-x86
I disagree, because this is the EBUILD HOWTO and hence only deals with gentoo-x86 and there no binaries are allowed at all. That would rather have a place in http://www.gentoo.org/doc/en/cvs-tutorial.xml but not here.
I don't see how to make it more clear: "Also, you should not add binary (non-ASCII) files to the Portage CVS tree". There is nothing to fix here.