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

adding qpoases and blocksqp #22

Closed
wants to merge 4 commits into from
Closed

Conversation

pstjohn
Copy link
Contributor

@pstjohn pstjohn commented May 9, 2019

Checklist

  • Used a fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

Adding build arguments for qpoases and blocksqp, addresses #21

@conda-forge-linter
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@pstjohn
Copy link
Contributor Author

pstjohn commented May 10, 2019

this is failing on the test with a mumps error. It's odd, since those same packages are installed during the build phase without issue

    mumps:           5.0.1-0                 conda-forge
    mumps-include:   5.1.2-1007              conda-forge
    mumps-seq:       5.1.2-h0ae83f2_1007     conda-forge
    ncurses:         6.1-hf484d3e_1002       conda-forge
    numpy:           1.16.3-py36he5ce36f_0   conda-forge
    openblas:        0.2.18-6                conda-forge
    openssl:         1.1.1b-h14c3975_1       conda-forge
    python:          3.6.7-h381d211_1004     conda-forge
    readline:        7.0-hf8c457e_1001       conda-forge
    scotch:          6.0.6-h491eb26_1002     conda-forge
    sqlite:          3.26.0-h67949de_1001    conda-forge
    tk:              8.6.9-h84994c4_1001     conda-forge
    xz:              5.2.4-h14c3975_1001     conda-forge
    zlib:            1.2.11-h14c3975_1004    conda-forge

Preparing transaction: ...working... done
Verifying transaction: ...working... done
Executing transaction: ...working... ERROR:conda.stderr:
ClobberWarning: This transaction has incompatible packages due to a shared path.
  packages: conda-forge::mumps-include-5.1.2-1007, conda-forge::mumps-5.0.1-0
  path: 'include/cmumps_c.h'

@pstjohn
Copy link
Contributor Author

pstjohn commented May 14, 2019

@conda-forge-admin, please rerender

@pstjohn
Copy link
Contributor Author

pstjohn commented May 17, 2019

Looks like I'm getting syntax errors on the packaging step. These might be issues with the generated python code?

For python3 builds:

Packaging casadi
INFO:conda_build.build:Packaging casadi
Packaging casadi-3.4.5-py37hbd7caa9_1004
INFO:conda_build.build:Packaging casadi-3.4.5-py37hbd7caa9_1004
compiling .pyc files...
  File "lib/python3.7/site-packages/casadi/tools/structure.py", line 271
    raise new_exc.__class__, new_exc, tb
                           ^
SyntaxError: invalid syntax

and for python 2 builds:

Packaging casadi
INFO:conda_build.build:Packaging casadi-3.4.5-py27h09a28d5_1004
Packaging casadi-3.4.5-py27h09a28d5_1004
compiling .pyc files...
  File "lib/python2.7/site-packages/casadi/tools/structure3.py", line 274
    raise Exception("Error occured in entry context with powerIndex %s, at canonicalIndex %s" % (str(powerIndex),str(canonicalIndex))) from e
                                                                                                                                          ^
SyntaxError: invalid syntax

@pariterre
Copy link
Contributor

@pstjohn
Do you want to keep this PR open?

@pstjohn
Copy link
Contributor Author

pstjohn commented Mar 4, 2020

I can close it, I don't really have the time to work on this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants