Skip to content

[ci skip]new package: openfoam2512#61092

Open
muezabdalla wants to merge 5 commits into
void-linux:masterfrom
muezabdalla:2512
Open

[ci skip]new package: openfoam2512#61092
muezabdalla wants to merge 5 commits into
void-linux:masterfrom
muezabdalla:2512

Conversation

@muezabdalla

Copy link
Copy Markdown

this is the openfoam the .com version. here is the .org (foundation) version

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for my native architecture, (x86_64-glibc)

@tranzystorekk tranzystorekk added the new-package This PR adds a new package label Jun 18, 2026
Comment thread srcpkgs/openfoam2512/template Outdated
hostmakedepends="wget"
makedepends="libfl-devel openmpi-devel gnuplot libboost_thread flex boost-devel openmpi"
depends="openmpi libboost_thread flex boost libgomp gnuplot"
short_desc="OpenFOAM is a free, open source CFD software released and developed by OpenCFD Ltd since 2004"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
short_desc="OpenFOAM is a free, open source CFD software released and developed by OpenCFD Ltd since 2004"
short_desc="Computational Fluid Dynamics (CFD) Toolbox"

For the description

  • it doesn't matter who develops it and since when
  • it doesn't matter if it's open source (if not, it would be in the nonfree repository anyways)
  • it's useless to repeat the name of the package, because the user already can see the package name anyways :)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

but there is 2 branches/forks of openfoam one is developed by OpenCFD and the other one is by the foundation. and the users will know which package is for which version with the description. and this description is what they have in there own website

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Im this case it's enough to add "developed by OpenCFD" to the description, the "2004" is still not needed :)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

edited it now

Comment thread srcpkgs/openfoam2512/template Outdated
Comment on lines +5 to +7
build_style=""
configure_args=""
hostmakedepends="wget"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
build_style=""
configure_args=""
hostmakedepends="wget"

Comment thread srcpkgs/openfoam2512/template Outdated
Comment on lines +20 to +31
do_fetch() {
mkdir -p $wrksrc
cd $wrksrc
wget ${distfiles}
}
do_extract() {
vtar -xf "OpenFOAM-${version}.tgz"
vtar -xf "ThirdParty-${version}.tar.gz"
mv "ThirdParty-${version}" "OpenFOAM-${version}/thirdparty"
}
do_build() {
cd "OpenFOAM-${version}"

@Bnyro Bnyro Jul 4, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why are you downloading and extracting the package yourself? xbps-src does that automatically for you

Suggested change
do_fetch() {
mkdir -p $wrksrc
cd $wrksrc
wget ${distfiles}
}
do_extract() {
vtar -xf "OpenFOAM-${version}.tgz"
vtar -xf "ThirdParty-${version}.tar.gz"
mv "ThirdParty-${version}" "OpenFOAM-${version}/thirdparty"
}
do_build() {
cd "OpenFOAM-${version}"
do_build() {

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

when I do this suggestion and the previus one I get this error:

=> openfoam2512-v2512_1: fetching distfile 'OpenFOAM-v2512.tgz' from 'https://dl.openfoam.com/source/v2512/OpenFOAM-v2512.tgz'...
ERROR: xbps-fetch: failed to fetch: https://dl.openfoam.com/source/v2512/OpenFOAM-v2512.tgz: Moved Temporarily
=> ERROR: openfoam2512-v2512_1: failed to fetch 'OpenFOAM-v2512.tgz'.

can you test these suggestion ?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please use ${SOURCEFORGE_SITE}/openfoam/OpenFOAM-v${version}.tgz, that's where it redirects to.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

thanks alot that worked

Comment thread srcpkgs/openfoam2512/template Outdated
Co-authored-by: Bnyro <bnyro@tutanota.com>
Comment thread srcpkgs/openfoam2512/template Outdated
@@ -0,0 +1,38 @@
# Template file for 'openfoam2512'
pkgname=openfoam2512
version=v2512

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The v shouldn't be part of the version.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

done

@@ -0,0 +1,38 @@
# Template file for 'openfoam2512'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The package sbould be called openfoam, not openfoam2512.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

the users usually would install mutiple versions of openfoam (e.g 2512, 2412, 2606). and they expect that the 2512 version will not get updated automatically to 2606.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-package This PR adds a new package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants