On Wed, Sep 23, 2009 at 8:37 AM, Jack Kelly <endgame....@gmail.com> wrote: > Hi Ralf, > > Thanks for the review. I agree with most of your remarks. > > On Wed, Sep 23, 2009 at 6:42 AM, Ralf Wildenhues <ralf.wildenh...@gmx.de> > wrote: >> * Jack Kelly wrote on Tue, Sep 22, 2009 at 04:33:27AM CEST: >>> + tagvars and verbose vars. >>> + * automake.in (define_verbose_tagvar): increase spacing to 12 to >>> + accommodate MAKEINFOHTML. >> >> Ugh, that's very ugly, because it will make it very difficult to align >> the target name with the abbreviation used for the command, when the >> build is rushing by. I found 6 to be close to the maximum tolerable >> length. Can we either agree to create unaligned output, or abbreviate >> MAKEINFOHTML somehow? > > Can we go to 8? If we have 8, I can change this to `INFOHTML' and it > means `MAKEINFO' is still OK.
Another thought: There's currently 2 spaces of padding that are unconditionally output. If we cut those when the tag is more than 6 characters long, the output would look like: MAKEINFO foo.info CC bar.o CCLD bar which would look OK but keep the 6 character limit on tags. What are your thoughts? -- Jack
