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
fix firefox sync-server #52654
fix firefox sync-server #52654
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two packages are missing a meta
section. Please add one.
Also, add a pythonPackages.
prefix to the commit messages.
applied the suggested changes |
propagatedBuildInputs = [ | ||
pyramid sqlalchemy simplejson mozsvc cornice pyramid_hawkauth pymysql | ||
pymysqlsa umemcache WSGIProxy requests pybrowserid | ||
pymysqlsa umemcache requests pybrowserid | ||
webtest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If webtest
is a test dependency, why is it here anyway?
broken = true; # 2018-11-04 | ||
postPatch = '' | ||
# conflict between WSGIProxy & WSGIProxy2 required by webtest | ||
# (which is required by pyramid_hawkauth) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This comment does not make sense
doCheck = false; # lazy packager | ||
propagatedBuildInputs = [ pyramid simplejson konfig ]; | ||
|
||
meta = with stdenv.lib; { | ||
homepage = https://github.com/mozilla-services/mozservices; | ||
homepage = "https://github.com/mozilla-services/mozservices"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no quotes
meta = with stdenv.lib; { | ||
description = "Run-Your-Own Firefox Sync Server"; | ||
homepage = "https://github.com/mozilla-services/syncserver"; | ||
platforms = platforms.unix; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
platforms = platforms.unix; |
meta = with stdenv.lib; { | ||
description = "The SyncServer server software, as used by Firefox Sync"; | ||
homepage = "https://github.com/mozilla-services/server-syncstorage"; | ||
platforms = platforms.unix; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
platforms = platforms.unix; |
This is set automatically by buildPython*
.
|
||
# since we disabled tests (which would need WSGIProxy) | ||
# lets get rid of the rest of the test dependencies as well: | ||
sed -i "s/unittest2.*//; s/webtest.*//; s/testfixtures.*//" requirements.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see a need for patching requirements.txt
. Then you can use substituteInPlace
after all.
I have tried to run it on my server, and it compiles, but running |
I have made a PR that fixes this: #54739. |
great, then this one can be closed |
Motivation for this change
pythonPackages.serversyncstorage
has been broken,this PR fixes it by disabling tests and removing a conflicting WSGIProxy dependency only used by tests
tested running
services.firefox.syncserver
, the service starts and shows the 'it works' status pagefixes #38830
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)