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 package: CalculiX-2.20 #39984
New package: CalculiX-2.20 #39984
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.
just a couple of things i noticed. there's a lot of work to be done here, I think, but I need to look into this more
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
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.
ok I think that's everything that I can see
srcpkgs/tetgen/template
Outdated
build_style="gnu-makefile" | ||
make_cmd="make -C ${pkgname}${version} -f makefile" |
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.
build_style="gnu-makefile" | |
make_cmd="make -C ${pkgname}${version} -f makefile" | |
build_style=cmake |
the cmake files provided are a bit better than the makefile
hostmakedepends+="glibc-devel-32bit" | ||
fi | ||
CFLAGS=" -O2 -Wall -Wno-narrowing -I./ -I/usr/include/ -I/usr/include/GL -I../../libSNL/src -I../../glut-3.5/src -I/usr/X11/include" | ||
|
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.
do_build() { | |
make CC=$CC CXX=$CXX CFLAGS="$CFLAGS" | |
} | |
Thank you for all your comments! I marked the PR as draft until I test all these modifications. |
6a3009d
to
4f2f0a3
Compare
I think the packages are ready for review!
I took all your comments into account (with one exception: I still use I also added a |
it's already in there: void-packages/etc/defaults.conf Lines 36 to 38 in fedd110
|
Actually I used a variable named I will fix this and use (I keep this last change as a separate commit, I'll rebase if the change is okay.) |
Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it. |
Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it. |
NEW PACKAGE - CalculiX
This new package conforms to the package requirements: YES
CalculiX is a 3D finite element program. It is an optional but important dependency of FreeCAD (for mechanical and thermal calculations). This PR is related to issue #5603.
CalculiX is split in two parts: the solver (CalculiX-ccx) and the graphical interface and pre/post-processing (CalculiX-cgx).
This PR includes 4 packages, each in a separate commit:
Tests
Local build testing
Cross build testing (from x86_64 machine)
./xbps-src binary-boostrap i686
on an x86_64 system, otherwise it fails)Runtime testing