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

Initial Windows build attempt #168

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

Conversation

mkitti
Copy link
Contributor

@mkitti mkitti commented Jan 5, 2022

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • 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.

@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.

@mkitti
Copy link
Contributor Author

mkitti commented Jan 5, 2022

@conda-forge-admin, please rerender

@mkitti
Copy link
Contributor Author

mkitti commented Jan 5, 2022

@conda-forge-admin, please restart ci

@mkitti
Copy link
Contributor Author

mkitti commented Jan 5, 2022

Mamba failed to solve:
 - vc >=14.1,<15.0a0
 - libgit2 1.3.*
 - suitesparse 5.*
 - libssh2 1.*
 - mbedtls
 - mpfr 4.*
 - p7zip
 - zlib 1.2.*
 - libnghttp2
 - libflang >=5.0.0,<6.0.0.a0
 - vs2015_runtime >=14.16.27012
 - libutf8proc
 - arpack 3.7.*
 - curl 7.*
 - gmp 6.*
 - git
 - pcre2 10.37.*
 - openblas-ilp64
 - openlibm

with channels:

The reported errors are:
- Encountered problems while solving:
-   - nothing provides requested mbedtls
-   - nothing provides requested p7zip
-   - nothing provides requested libnghttp2
-   - nothing provides requested gmp 6.*
-   - nothing provides requested openblas-ilp64

@mkitti
Copy link
Contributor Author

mkitti commented Jan 5, 2022

make: Entering directory 'D:/bld/julia_1641366472651/work/base'
D:/bld/julia_1641366472651/work/Make.inc:135: *** cowardly refusing to build into directory with a shell-metacharacter in the path (got: D:/bld/julia_1641366472651/work).  Stop.
make: Leaving directory 'D:/bld/julia_1641366472651/work/base'
D:/bld/julia_1641366472651/work/Make.inc:135: *** cowardly refusing to build into directory with a shell-metacharacter in the path (got: D:/bld/julia_1641366472651/work).  Stop.

https://github.com/JuliaLang/julia/blob/85a6990a9c1d49dd5aeaffeb4b38f881dc120823/Make.inc#L130-L144

@mkitti
Copy link
Contributor Author

mkitti commented Jan 8, 2022

Unless someone can tell me how to use msys2 or cygwin to build, this is probably dead.

xref: conda-forge/conda-forge.github.io#112

@ngam
Copy link
Contributor

ngam commented Jan 8, 2022

Could you rebase this with master?

@mkitti
Copy link
Contributor Author

mkitti commented Jan 8, 2022

There's nothing to rebase. This approach is not going to work.

@mkitti
Copy link
Contributor Author

mkitti commented Jan 9, 2022

The best I can think of is grab the m2w64 packages, and just build a fully vendored Julia to start.

@ngam
Copy link
Contributor

ngam commented Jan 9, 2022

😞

(full disclosure: I have zero idea how windows works --- haven't used windows in ~15 years; last month I installed windows11 and I couldn't install docker... I couldn't even install conda, so after a few hours, I deleted it)

@mkitti
Copy link
Contributor Author

mkitti commented Feb 1, 2022

Hmm... the gmp feedstock looks kind of interesting...

https://github.com/conda-forge/r-gmp-feedstock/blob/master/recipe/meta.yaml#L24-L27

@conda-forge-linter
Copy link

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

I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.

Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

@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.

@mkitti
Copy link
Contributor Author

mkitti commented Feb 18, 2022

https://anaconda.org/conda-forge/m2-bash is probably better.

We may also need to explicitly put python on the path somehow
@conda-forge-linter
Copy link

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

I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.

Here's what I've got...

For recipe:

  • Selectors are suggested to take a <two spaces>#<one space>[<expression>] form. See lines [27]
  • If python is a host requirement, it should be a run requirement.

@conda-forge-linter
Copy link

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

I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.

Here's what I've got...

For recipe:

  • If python is a host requirement, it should be a run requirement.

recipe/bld.bat Outdated
set MSYSTEM=MINGW%ARCH%
set MSYS2_PATH_TYPE=inherit
set CHERE_INVOKING=1
bash -lc ./build.sh
Copy link
Contributor

Choose a reason for hiding this comment

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

Interesting approach... it doesn't seem it is cooperating?

I would think rewriting build.sh here would be the way to go, but I have no idea how windows works. One thing I would try to start off would be to just copy the binaries from upstream and see if that works with a simple bld.bat to copy them to PREFIX. Then I would try to actually build...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thr Julia build process essentially needs bash on Windows. It usually uses Cygwin and has been used with msys2 in the past. The current issue is locating Python on the path.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually the problem seems different now. I wonder why we do not see any evidence of building.

@mkitti mkitti closed this Apr 6, 2022
@mkitti mkitti reopened this Apr 6, 2022
@conda-forge-linter
Copy link

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

I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.

Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

@conda-forge-webservices
Copy link

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

I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.

Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

@conda-forge-webservices
Copy link

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

I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.

Here's what I've got...

For recipe:

  • If python is a host requirement, it should be a run requirement.

@conda-forge-webservices
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.

@mkitti
Copy link
Contributor Author

mkitti commented Jul 18, 2023

Hmm, did I just miss call in the BAT file?

@mkitti
Copy link
Contributor Author

mkitti commented Jul 18, 2023

@conda-forge-admin, please rerender

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.

3 participants