bug-wget
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-wget] Travis-CI updates and minor bug fixes


From: Darshit Shah
Subject: Re: [Bug-wget] Travis-CI updates and minor bug fixes
Date: Mon, 12 Oct 2015 02:02:45 +0200

I have attached an updated set of patches. The ChangeLog between the
last set and this is:

1. Fix missing $srcdir in proxied-https tests.
2. Remove extra CFLAGS env var from .travis.yml
3. Add extra suppressions to allow the test to pass on travis

However, the build still fails on Travis. It fails because the
assertion in the progress bar fires each time during the Turkish
locale tests. As it turns out, the translators decided not to heed the
specific instruction provided to them to keep the translation for ETA
to 3 columns only. As a result, the length of the progress bar exceeds
the screen width, causing the assertion to fire. I think we should
talk to the translators about this issue and ask them to fix it.


Apart from that, of there are no issues with this set of patches, I
will push them sometime later today.
Currently, a build on travis will be fired when a commit is pushed on
my GitHub clone of Wget. I will look into the POST API later. Maybe
some kind of a local git hook may work.

On 11 October 2015 at 17:18, Tim Rühsen <address@hidden> wrote:
> Am Sonntag, 11. Oktober 2015, 12:32:14 schrieb Darshit Shah:
>> However, I did come across a small problem. Travis currently accepts
>> open source build requests from GitHub only. So I don't think we can
>> use it from Savannah. Would it be a problem if we pushed the
>> .travis.yml to master and allow the builds to be fired from one of our
>> GitHub clones?
>
> Hi Darshit,
>
> IMO there is no problem.
>
> If there is no possibility to trigger a build from Savannah, we have to use
> Github. Or we have to trigger it manually...
>
> That is something that other people already asked for:
> http://kamranicus.com/blog/2015/03/29/triggering-a-travis-build-programmatically/
>
> Maybe you could give this a try (maybe using wget instead of curl ?).
> http://docs.travis-ci.com/user/triggering-builds/
>
> Regards, Tim
>



-- 
Thanking You,
Darshit Shah

Attachment: 0004-Add-suppressions-for-Valgrind-3.4-bug.patch
Description: Text Data

Attachment: 0003-Add-.travis.yml-file-and-script-for-Travis-CI.patch
Description: Text Data

Attachment: 0002-Fix-Test-ftp-pasv-not-supported.px.patch
Description: Text Data

Attachment: 0001-Fix-make-distcheck-failures-in-Perl-SSL-Tests.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]