Skip to content
This repository has been archived by the owner on Oct 7, 2022. It is now read-only.

Help packaging azure-cli #194

Closed
colemickens opened this issue May 8, 2018 · 3 comments
Closed

Help packaging azure-cli #194

colemickens opened this issue May 8, 2018 · 3 comments

Comments

@colemickens
Copy link
Member

colemickens commented May 8, 2018

Hello.

I'm trying to package azure-cli for NixOS with an overlay using pypi2nix.

The very latest version(s) seem to rely on pip-10.0 (hence me filing #192) but I'm not sure they would've actually broken compatibility with older versions of pip, especially since I'm using requirements.txt.

Here's what I'm running:

#!/usr/bin/env bash

set -euo pipefail
set -x

docker run -it "microsoft/azure-cli-python:2.0.32" pip freeze \
    | head -n -2 > requirements.txt

sudo rm -rf /tmp/pypi2nix

# TODO: pretty sure this shouldn't need root...
sudo pypi2nix -v \
    --python-version "3.6" \
    --extra-build-inputs "gcc" \
    --extra-build-inputs "libffi" \
    --extra-build-inputs "openssl" \
    --requirements "./requirements.txt" \
        | tee "log.txt"

Here's the output: https://gist.github.com/d26b7f152e80d1eb0902ff45c9b6e419

Research:

I tried adding wheel as a --setup-requires but it fails from a ZIP 1970 error and I'm not sure it is the right fix anyway.

Any suggestions would be appreciated. Thanks!

@garbas
Copy link
Contributor

garbas commented May 9, 2018

@colemickens hey, i don't have time right now to look deeper into this, all i can promise is to come back to this in 2 weeks (if this is still the problem).

@layus
Copy link

layus commented Nov 19, 2018

@colemickens @garbas Maybe have a look at #238.

It seems we need to use the bdist_wheel wrapper provided by azure_bdist_wheel.py so that 'azure-namespace-package' option gets recognized.
Not too sure how to achieve that, because we explicitly do not install the wheel package (IIUC).

@garbas
Copy link
Contributor

garbas commented Dec 2, 2018

currently the problem is that azure-cli does not support latest wheel package and downgrading it is not yet possible.

@garbas garbas closed this as completed Dec 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants