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: Wed, 18 Nov 2015 00:17:41 +0100
User-agent: Mutt/1.5.24+27 (c799162691b2) (2015-08-30)

It's been a while since I shared these patches. The fixes to make distcheck went in before the last release, however, we still don't have Travis integration. The API Tim linked to is valid only for private builds which is a paid feature of Travis.

Since the Turkish tests were failing due to a bad translation, I've currently removed it from the list of languages. Once we have the translation fixed, we can add the Turkish language back into the tests. I'd like to have that language in because of the specific corner case provided in its character set.

Apart from that, I randomly picked and added Japanese against a language to test against. This was chosen because it has multi-byte and multi-column characters and I want to ensure that the progress bar works perfectly for these. Turns out in one particular case it doesn't. Which is why the current Travis setup fails too. Hence, I removed the language for now so that we have a working Travis setup. However, that was a valid bug and we need to look into fixing it.

Currently, the only build that fails is one with --without-ssl. While I understand that this is a ridiculous idea in this day, but we provide a configure option to build without SSL and should support it. I've left it there since it's a more major issue we need to look into before the next minor release.

Email notifications from Travis are not currently working. I will discuss this on IRC with them tomorrow to see how we can fix this.

I've attached the new patch for Travis integration and will push it if no one complains.

Eventually, I'd like to extend this travis file to cover OSX builds too. But currently, the OSX environment seems to have some problems with Python that requires some manual work. Hence, I've not included the OSX as a target OS for testing in this patch.



On 10/11, Tim Rühsen 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: 0001-Introduce-Travis-Integration.patch
Description: Text Data

Attachment: signature.asc
Description: PGP signature


reply via email to

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