Closed Bug 1371707 Opened 7 years ago Closed 7 years ago

Fix Travis dependencies for Linux x86

Categories

(Testing :: geckodriver, enhancement)

Version 3
x86
Linux
enhancement
Not set
normal

Tracking

(firefox56 fixed)

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: ato, Assigned: ato)

Details

Attachments

(1 file)

      No description provided.
Assignee: nobody → ato
Status: NEW → ASSIGNED
Can you please link to the failure?
Flags: needinfo?(ato)
(In reply to Henrik Skupin (:whimboo) from comment #2)
> Can you please link to the failure?

https://travis-ci.org/mozilla/geckodriver/jobs/241215030
Flags: needinfo?(ato)
Comment on attachment 8876184 [details]
Bug 1371707 - Fix Linux x86 packages on Travis for geckodriver;

https://reviewboard.mozilla.org/r/147600/#review152870

::: testing/geckodriver/.travis.yml:13
(Diff revision 1)
>  before_install:
>    - if [[ "$NAME" == "linux32" ]]; then
>      sudo dpkg --purge build-essential libtool;
>      sudo apt-get -qq update;
> -    sudo apt-get -yq --force-yes install libstdc++6:i386 libbz2-dev:i386 gcc-multilib:i386 musl-tools:i386; fi
> -    
> +    sudo apt-get -yq --force-yes install libstdc++6:i386 libbz2-dev:i386 gcc-4.8:i386 gcc-4.8-multilib:i386 musl-tools:i386 binutils:i386; fi
> +

It looks like you hit https://github.com/travis-ci/travis-ci/issues/5670 here. Does updating `gcc` first as described in this issue fixes the problem?

I would love to have the virtual packages listed here, so that we do not have to update this once in a while.

But if this code only exists until we can build everything in tree via Taskcluster, we might wanna get this landed as is.
Comment on attachment 8876184 [details]
Bug 1371707 - Fix Linux x86 packages on Travis for geckodriver;

https://reviewboard.mozilla.org/r/147600/#review152870

> It looks like you hit https://github.com/travis-ci/travis-ci/issues/5670 here. Does updating `gcc` first as described in this issue fixes the problem?
> 
> I would love to have the virtual packages listed here, so that we do not have to update this once in a while.
> 
> But if this code only exists until we can build everything in tree via Taskcluster, we might wanna get this landed as is.

I would quite like to get this landed as-is.  Maintaining the Travis build environment is not a priority, and I don’t have any extra time to spend on this.  If you do, you’re welcome to follow up on this and submit a patch.
Comment on attachment 8876184 [details]
Bug 1371707 - Fix Linux x86 packages on Travis for geckodriver;

https://reviewboard.mozilla.org/r/147600/#review152870

> I would quite like to get this landed as-is.  Maintaining the Travis build environment is not a priority, and I don’t have any extra time to spend on this.  If you do, you’re welcome to follow up on this and submit a patch.

Sure. And I still assume it's only used for the moment until we can build all in tree.
Comment on attachment 8876184 [details]
Bug 1371707 - Fix Linux x86 packages on Travis for geckodriver;

https://reviewboard.mozilla.org/r/147600/#review152912
Attachment #8876184 - Flags: review?(hskupin) → review+
(In reply to Henrik Skupin (:whimboo) from comment #6)

> > I would quite like to get this landed as-is.  Maintaining the Travis
> > build environment is not a priority, and I don’t have any extra
> > time to spend on this.  If you do, you’re welcome to follow up on
> > this and submit a patch.
> 
> Sure. And I still assume it's only used for the moment until we can
> build all in tree.

Yes, precisely.  I want us to get to the point where we can
release to GitHub from in-tree, but we are currently lacking
some platforms and some infrastructure to do that.  I filed
https://bugzilla.mozilla.org/show_bug.cgi?id=1372587 to track this.
Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/41d5632628a6
Fix Linux x86 packages on Travis for geckodriver; r=whimboo
https://hg.mozilla.org/mozilla-central/rev/41d5632628a6
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: