Mailing List Archive

Vtest compiler/arch issue
I spent some time looking at this.

I want to avoid, to the fullest extent possible, that people have
to supply such information by hand, because they generally suck at
it, and we should be able to autodetect the compiler/target/architecture.

This patch will do it: (NB: Copy&Pasted)

diff --git a/bin/varnishtest/Makefile.am b/bin/varnishtest/Makefile.am
index 26563e2..6b7e4eb 100644
--- a/bin/varnishtest/Makefile.am
+++ b/bin/varnishtest/Makefile.am
@@ -9,7 +9,15 @@ check: check-am check-local
check-am: check-local
# See if list of checks have changed, recheck
check-local:
- LC_ALL=C; \
+ @echo "VTEST CCVER BEGIN"
+ @ for i in --version -v -V -qversion ; \
+ do \
+ if ${CC} ${i} ; then \
+ break ; \
+ fi \
+ done
+ @echo "VTEST CCVER END"
+ @LC_ALL=C; \
if [ "$$(cd $(srcdir) && echo tests/*.vtc)" != "@VTC_TESTS@" ]; then \
cd $(top_builddir) && ./config.status --recheck ; \
fi

Only, that doesn't work at all in practice.

I need to be able to build the "id" of the report also if the
"autogen" step fails, which is where the configure script
which eventually determins what compiler and target arch
will be.

Also, if makedistcheck ends up GOOD, the output does not get returned.

So I think we are stuck with manual input for this and maybe other
cases as well.

So I'm going to add a "message" variable in vtest.sh which the owner
of the machine can set, and that will be part of the 'id' and
displayed at the bottom line at the table header

People get to write whatever they want in that message, provided
they use max 10 char from the set [A-Za-z0-9/. _-].

--
Poul-Henning Kamp | UNIX since Zilog Zeus 3.20
phk@FreeBSD.ORG | TCP/IP since RFC 956
FreeBSD committer | BSD since 4.3-tahoe
Never attribute to malice what can adequately be explained by incompetence.

_______________________________________________
varnish-dev mailing list
varnish-dev@varnish-cache.org
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev
Re: Vtest compiler/arch issue [ In reply to ]
> People get to write whatever they want in that message, provided
> they use max 10 char from the set [A-Za-z0-9/. _-].

I'm feeling compelled to point out that yesterday you were complaining
about artificial compiler limitations that are no longer relevant in
today's computing. And as such with 10 characters I can't even fit
"Linux-x86_64":

MESSAGE=$(uname -s)-$(uname -m)

Cheers,
Dridi

_______________________________________________
varnish-dev mailing list
varnish-dev@varnish-cache.org
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev
Re: Vtest compiler/arch issue [ In reply to ]
--------
In message <CABoVN9DntoZjSPmQduu=8wmOGidMAKa5qP6d9LENuORuux2J_g@mail.gmail.com>, Dridi Boukelmoune writes:
>> People get to write whatever they want in that message, provided
>> they use max 10 char from the set [A-Za-z0-9/. _-].
>
>I'm feeling compelled to point out that yesterday you were complaining
>about artificial compiler limitations that are no longer relevant in
>today's computing.

Correct.

But that has absolutly nothing to do with putting random
user input into your HTML web-pages without proper data-hygiene.

>today's computing. And as such with 10 characters I can't even fit
>"Linux-x86_64":

You wouldn't need to, that is already derived from the uname and lsb
information.

This is for writing "gcc-m32" or similar.


--
Poul-Henning Kamp | UNIX since Zilog Zeus 3.20
phk@FreeBSD.ORG | TCP/IP since RFC 956
FreeBSD committer | BSD since 4.3-tahoe
Never attribute to malice what can adequately be explained by incompetence.

_______________________________________________
varnish-dev mailing list
varnish-dev@varnish-cache.org
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev