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

New packages: pastix4calculix #8018

Closed
wants to merge 3 commits into from
Closed

New packages: pastix4calculix #8018

wants to merge 3 commits into from

Conversation

3rav
Copy link
Contributor

@3rav 3rav commented Feb 24, 2021

No description provided.

@Biswa96
Copy link
Member

Biswa96 commented Feb 24, 2021

Python2? There are discussions about getting rid of it! The provides should have MINGW_PACKAGE_PREFIX also.

@3rav
Copy link
Contributor Author

3rav commented Feb 24, 2021

With the another version (I mean 3) I have a bug preventing compilation.

@lazka
Copy link
Member

lazka commented Feb 24, 2021

yeah, no more python2 please.

@3rav
Copy link
Contributor Author

3rav commented Feb 25, 2021

I have this error with python3:

-- Generating done
  CMake Error:
    Running
  
     'D:/a/_temp/msys/msys64/mingw64/bin/ninja.exe' '-C' 'C:/_/mingw-w64-pastix4calculix/src/build-x86_64-w64-mingw32-static' '-t' 'recompact'
  
    failed with:
  
     ninja: error: build.ninja:1842: expected '=', got lexing error
  
    common/z_integer.c" -p s -P common && chmod a-w common/s_integer.c"
  
  
          ^ near here
  
  
  
  CMake Generate step failed.  Build files cannot be regenerated correctly.
  ==> ERROR: A failure occurred in build().
      Aborting...

@Biswa96
Copy link
Member

Biswa96 commented Feb 25, 2021

In the patch file, what is wrong with all those memset?

@3rav
Copy link
Contributor Author

3rav commented Feb 25, 2021

Previously, instead of memset was bzero, I made "pull requests" for this (Kabbone/PaStiX4CalculiX#1) but now I decided that maybe it is better to unify the syntax of the main official version 6.1.0 Pastix: https://gitlab.inria.fr/solverstack/pastix/-/blob/master/order/order_find_supernodes.c

@Biswa96
Copy link
Member

Biswa96 commented Feb 25, 2021

Just curious. Why is a forked repository (Kabbone/PaStiX4CalculiX) used instead of actual one (Dhondtguido/PaStiX4CalculiX)?

@3rav
Copy link
Contributor Author

3rav commented Feb 25, 2021

Dhondtguido/PaStiX4CalculiX requires CUDA but this version does not (Kabbone/PaStiX4CalculiX).
Dhondtguido/PaStiX4CalculiX#1 (comment)

@3rav
Copy link
Contributor Author

3rav commented Mar 4, 2021

Python2? There are discussions about getting rid of it! The provides should have MINGW_PACKAGE_PREFIX also.

Where can I find this discussion?

@3rav 3rav closed this Apr 12, 2021
@3rav 3rav deleted the patch-1 branch April 12, 2021 07:02
@3rav 3rav restored the patch-1 branch April 13, 2021 04:21
@3rav 3rav deleted the patch-1 branch October 25, 2022 06:19
This was referenced Jan 5, 2024
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

3 participants