Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add package for pylibmc, adding cyrus_sasl to libmemcached's propagatedB... #7055

Closed
wants to merge 1 commit into from

Conversation

risicle
Copy link
Contributor

@risicle risicle commented Mar 28, 2015

...uildInputs while we're at it

@risicle
Copy link
Contributor Author

risicle commented Mar 29, 2015

Hmm could someone with more experience of the travis system tell me what's failing here?

@@ -15,6 +15,8 @@ stdenv.mkDerivation rec {

buildInputs = [ cyrus_sasl libevent ];

propagatedBuildInputs = [ cyrus_sasl ];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should then be removed from buildInputs

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahhh I must be going blind.

@risicle
Copy link
Contributor Author

risicle commented Apr 1, 2015

mmm...?

It can be automatically merged, but the merge failed?

...in a file I didn't touch? (pkgs/tools/misc/cpulimit/default.nix)

@risicle
Copy link
Contributor Author

risicle commented Apr 1, 2015

Travis is sucking at the moment (download speeds of 17KBps?) so this may pass if someone retries it.

@nbp
Copy link
Member

nbp commented Nov 16, 2015

cc @domenkozar

@domenkozar
Copy link
Member

My bad @risicle can you rebase on top of master?

@risicle
Copy link
Contributor Author

risicle commented Mar 13, 2016

^Travis log length exceeded and I don't currently have access to a machine that can do a local build.

@risicle
Copy link
Contributor Author

risicle commented Mar 26, 2016

ok, ^a nox build for this works for me locally (though i bet travis won't work)

@risicle
Copy link
Contributor Author

risicle commented Mar 29, 2016

@domenkozar poke

@7c6f434c
Copy link
Member

I don't understand what has happened here, but as of 16fff5e the master should contain all of these changes.

@7c6f434c 7c6f434c closed this Mar 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants