[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Autobuild-discuss] A few nits with autobuild
From: |
Simon Josefsson |
Subject: |
Re: [Autobuild-discuss] A few nits with autobuild |
Date: |
Sat, 27 Aug 2011 12:23:47 +0200 |
User-agent: |
Gnus/5.110018 (No Gnus v0.18) Emacs/23.2 (gnu/linux) |
Julien ÉLIE <address@hidden> writes:
> Hi,
>
> I have started to test an autobuild page for INN:
> http://autobuild.josefsson.org/inn/
>
> I am really satisfied by the result. It is nice to have such a web page.
Hi and welcome! Glad you like it!
> Here are a few comments after a few days of use:
>
> 1/
> There is an issue with the value of HTML tokens:
> <h3><a name="inn-GCC 4.6.1">
>
> Spaces should be stripped, or converted to something else. (A dash?)
It seems to work in IceCat, are spaces really forbidden in links or
names?
> 2/
> Why are there all these
> <br />
> at the end of the HTML page? Couldn't they be removed from the
> autobuild Perl script? (I see it prints lots of them!)
Yeah, that is a bit of a hack. The reason is to make sure the last <a
name> link will focus at the top of the page, otherwise it is difficult
to find it. Compare these two links:
http://autobuild.josefsson.org/ccscript/#ccscript-gcc10
http://autobuild.josefsson.org/ccscript/#ccscript-%27
If there weren't some blanks at the end, it wouldn't focus correctly.
Is there a better solution for this?
> 3/
> doc/autobuild.texi
> --timestamp mentions a preferred `YYYY-MM-DD+HH.MM.SS' format.
> Yet, `YYYY-MM-DDTHH:MM:SS' is said to be preferred in the source code
> of the autobuild Perl script. I also believe this standardized format
> is better.
I agree. Fixed in git, thank you!
/Simon