Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#49231 closed enhancement (fixed)

OpenBLAS and OpenBLAS-devel: unify into one Portfile with subport

Reported by: dstrubbe (David Strubbe) Owned by: NicosPavlov
Priority: Normal Milestone:
Component: ports Version: 2.3.4
Keywords: haspatch Cc: petrrr
Port: OpenBLAS, OpenBLAS-devel

Description

The OpenBLAS and OpenBLAS-devel Portfiles are very similar. Only a few lines are different. I suggest unifying them into a single port with a subport, to make future maintainence simpler. Though these ports are openmaintainer, I guess this doesn't count as a minor change, so I submit it for the maintainer's consideration. The patch of course would be accompanied by 'svn mv'ing the two files in OpenBLAS-devel/files which are not in OpenBLAS/files and then 'svn rm'ing OpenBLAS-devel.

Attachments (1)

Portfile-OpenBLAS.diff (2.1 KB) - added by dstrubbe (David Strubbe) 9 years ago.

Download all attachments as: .zip

Change History (9)

Changed 9 years ago by dstrubbe (David Strubbe)

Attachment: Portfile-OpenBLAS.diff added

comment:1 Changed 9 years ago by NicosPavlov

I understand the motivation, but the two ports are gradually becoming more different with time, as OpenBLAS is not releasing new versions very often, while several enhancements have been pushed to the repository. This implies that while *-devel is being updated often, bugfixes have to be included in the standard release. I am afraid that having the two of them in the same file could increase the risk of mistakes.

Furthermore, having the two ports separate in a case like this is, at least to my knowledge, standard practice within Macports.

comment:2 in reply to:  1 Changed 9 years ago by seanfarley (Sean Farley)

Replying to nicos@…:

I understand the motivation, but the two ports are gradually becoming more different with time, as OpenBLAS is not releasing new versions very often, while several enhancements have been pushed to the repository. This implies that while *-devel is being updated often, bugfixes have to be included in the standard release. I am afraid that having the two of them in the same file could increase the risk of mistakes.

Furthermore, having the two ports separate in a case like this is, at least to my knowledge, standard practice within Macports.

I would disagree. There is so much code duplication between the two portfiles that it definitely makes sense to combine them. The patches and backports are trivial to separate. Please reconsider.

comment:3 Changed 9 years ago by dstrubbe (David Strubbe)

There are example of both patterns: port and port-devel have separate Portfiles, or port-devel as subport in port's Portfile. mpich and openmpi are examples of the latter. So, I think both can be considered standard practices.

My motivation is I want to propose a reform of the treatment of variants, to use the compilers portgroup, and to use the logic that currently changes the meaning of the clang variant to instead set default variants accordingly, which would make things more transparent and controllable to the user. Since this change would apply equally to OpenBLAS and OpenBLAS-devel, it could be applied only once if the Portfiles were merged.

comment:4 Changed 9 years ago by seanfarley (Sean Farley)

Yes, what dstrubbe wrote is exactly what I have in mind for most -devel ports. Not only would variants be easier to unify but this could help pave the way for a blas/lapack portgroup.

comment:5 Changed 9 years ago by petrrr

What about the julia packaged OpenBlas? Should this be considered as well?

comment:6 Changed 9 years ago by petrrr

Cc: petr@… added

Cc Me!

comment:7 in reply to:  5 Changed 9 years ago by NicosPavlov

Resolution: fixed
Status: newclosed

I was not aware that using subports in that case was considered standard. Committed in r141428, with some tweaks to further minimize the differences.

Replying to petr@…:

What about the julia packaged OpenBlas? Should this be considered as well?

Julia is employing a specific interface of OpenBLAS, which is not the default one. It was to my understanding decided to leave the compilation internal for this reason. I had proposed in another ticket to add the option in the Portfile, but it did not lead to anything concrete yet.

I close the ticket as the issue is fixed, but I am still open for discussion about the integration with julia.

comment:8 Changed 9 years ago by seanfarley (Sean Farley)

As far as I'm aware, julia uses the 64-bit integer interface for OpenBLAS which would break any port using OpenBLAS today.

Note: See TracTickets for help on using tickets.