Skip to content

Perl: disable tests that involve networking#1905

Closed
pjotrp wants to merge 2 commits intoNixOS:masterfrom
pjotrp:perl
Closed

Perl: disable tests that involve networking#1905
pjotrp wants to merge 2 commits intoNixOS:masterfrom
pjotrp:perl

Conversation

@pjotrp
Copy link
Contributor

@pjotrp pjotrp commented Mar 9, 2014

These tests fail on my setup - building in user land on a SARA cluster node

@domenkozar
Copy link
Member

Looks more sensible than #1632

vcunat added a commit to vcunat/nixpkgs that referenced this pull request Mar 9, 2014
I rebased the branch not to include newer changes from master.
@vcunat
Copy link
Member

vcunat commented Mar 9, 2014

Yes, I'm adding this to my choice of simple stdenv-changing improvements proposed to be merged soon. vcunat/nixpkgs@NixOS:glibc-2.19...p/stdenv

vcunat added a commit to vcunat/nixpkgs that referenced this pull request Apr 2, 2014
I rebased the branch not to include newer changes from master.
@shlevy
Copy link
Member

shlevy commented Apr 5, 2014

@vcunat This should be fixed now, right?

@vcunat
Copy link
Member

vcunat commented Apr 5, 2014

Yes, it's in master already, only github doesn't accept the merge keyword, I guess.

@vcunat vcunat closed this Apr 5, 2014
lucabrunox pushed a commit to lucabrunox/nixpkgs that referenced this pull request Apr 6, 2014
I rebased the branch not to include newer changes from master.
@pjotrp
Copy link
Contributor Author

pjotrp commented Jun 3, 2014

The patch is still not in master, that I can tell. Why is that? Should I send a patch to the ML?

@vcunat
Copy link
Member

vcunat commented Jun 3, 2014

Here's when it was included: 1046b4cca

Even github claims it's in master now (if you click the commit, it shows a list of branches including it), and I verified that the two now lines are still present in master.

@edolstra
Copy link
Member

edolstra commented Jun 3, 2014

Note that I got rid of the Perl tests altogether: f95196a

@vcunat
Copy link
Member

vcunat commented Jun 3, 2014

Well, I don't know how much the tests can reveal. However, stdenv isn't rebuilt so often, and for me it's roughly 2-3 hours (a single cheap machine).

@pjotrp
Copy link
Contributor Author

pjotrp commented Jun 3, 2014

My mistake, sorry. git confusion can sometimes kicks in. All hunky dory now.Thanks! And +1 for removing all the Perl tests.

@pjotrp pjotrp deleted the perl branch June 4, 2014 11:41
@Janik-Haag Janik-Haag added the 12.first-time contribution This PR is the author's first one; please be gentle! label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

12.first-time contribution This PR is the author's first one; please be gentle!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants