Opened 13 years ago

Closed 12 years ago

#31650 closed submission (invalid)

rose @0.9.5a

Reported by: too1@… Owned by: neverpanic (Clemens Lang)
Priority: Normal Milestone:
Component: ports Version:
Keywords: lack-of-interest Cc: drkp (Dan Ports)
Port: rose

Description


Attachments (4)

Portfile (1.9 KB) - added by too1@… 13 years ago.
Portfile-rose.diff (1.2 KB) - added by too1@… 13 years ago.
Portfile.2 (2.2 KB) - added by neverpanic (Clemens Lang) 12 years ago.
config.log (18.6 KB) - added by neverpanic (Clemens Lang) 12 years ago.
config.log from buildTree/

Download all attachments as: .zip

Change History (14)

comment:1 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Thanks. Some comments:

  • Your email address should be obfuscated in the maintainers line: llnl.gov:too1.
  • The license should be just "BSD".
  • Please fetch from an official distfile. If none is available, fetch from the specific repository revision or branch that represents version 0.9.5a. Fetching from HEAD is not predictable.
  • When fetch.type is "git", MacPorts adds git-core as a fetch dependency for you; you don't need to add it manually.
  • The dependency on graphviz should be written so that graphviz-devel could satisfy it; see other ports that depend on graphviz for examples of how this is done, or #14540.
  • I notice the build dependency on wget... Does this software really use wget to fetch things during its build? Ports should not be fetching things during the build phase; they should fetch all necessary files during the fetch phase, if possible.
  • "gcc" is not a valid value for configure.compiler; you seem to want configure.compiler macports-gcc-4.4. Then, where you hardcode ${prefix}/bin/gfortran-mp-4.4, you can replace that with ${configure.fc}. And you can add compiler variants to give users a choice. See PortfileRecipes.
  • You probably shouldn't set build.jobs to 0, thereby overriding the user's choice of number of build jobs.
  • Note that system now has a -W argument that you can use instead of manually cd'ing into a directory: system -W ${worksrcpath} "./build"

Changed 13 years ago by too1@…

Attachment: Portfile added

comment:2 Changed 13 years ago by too1@…

Thanks for the feedback. I've attached a revised Portfile:

  • ROSE has been stuck at v0.9.5a for several years. The master branch would be the best option for our users until we start versioning properly (something we plan on doing soon). Maybe it would be better to have this port be rose-devel?
  • ROSE must be compiled with the system default compiler (GCC 4.2.1); so only gfortran-mp-4.4 is needed. Yes, it's a bit quirky to get ROSE installed (a big reason for the MacPort, to help our users install ROSE more easily).

comment:3 Changed 13 years ago by too1@…

Cc: too1@… added

Cc Me!

comment:4 Changed 13 years ago by neverpanic (Clemens Lang)

Keywords: haspatch added

comment:5 Changed 13 years ago by neverpanic (Clemens Lang)

Cc: cal@… added; too1@… removed

You don't need to Cc yourself.

comment:6 Changed 13 years ago by mf2k (Frank Schima)

Keywords: maintainer haspatch removed
Version: 2.0.3

comment:7 in reply to:  2 Changed 13 years ago by drkp (Dan Ports)

Cc: dports@… added

Replying to too1@…:

Thanks for the feedback. I've attached a revised Portfile:

  • ROSE has been stuck at v0.9.5a for several years. The master branch would be the best option for our users until we start versioning properly (something we plan on doing soon). Maybe it would be better to have this port be rose-devel?

Whether it's named -devel or not, we really need the port to install a fixed version of the software rather than checking out HEAD from git -- we need reproducible builds so that MacPorts knows what version a user has installed, and so that we can (possibly) distribute binaries. The usual way to do this would be to use a date or something as the version, and set git.branch to the corresponding commit hash (see libusb-devel for an example).

Other than that, the portfile basically looks reasonable to me...

Changed 13 years ago by too1@…

Attachment: Portfile-rose.diff added

comment:8 Changed 12 years ago by neverpanic (Clemens Lang)

Cc: cal@… removed
Owner: changed from macports-tickets@… to cal@…
Status: newassigned

comment:9 Changed 12 years ago by neverpanic (Clemens Lang)

I wanted to commit the Portfile with the following changes:

  • Added standard modeline
  • Used the github portgroup and github.setup instead of specifying the fetch form github manually
  • path:-style dependencies don't need ${prefix}
  • Changed github username to rose-compiler, since this apparently is the new home of rose
  • Added checksums, since the github portgroup downloads a tar.gz
  • Changed default compiler to gcc45, since that currently is our favorite gcc compiler

However, the build failed with:

checking for ROSE Git version information in '/opt/local/var/macports/build/_opt_dports_lang_rose/rose/work/rose-af169e174891539228704cb1befc8fdda4be94c7'... configure: error: in `/opt/local/var/macports/build/_opt_dports_lang_rose/rose/work/rose-af169e174891539228704cb1befc8fdda4be94c7/buildTree':
configure: error: /opt/local/var/macports/build/_opt_dports_lang_rose/rose/work/rose-af169e174891539228704cb1befc8fdda4be94c7/VERSION file does not exist
See `config.log' for more details

I'm attaching config.log and my modified Portfile. Note that since I use the github portgroup the sources are in fact not being fetched via git, but downloaded as a tarball and do not contain a complete git history. Maybe rose requires to be built from a git repository? Can this requirement be lifted by providing the VERSION file?

Changed 12 years ago by neverpanic (Clemens Lang)

Attachment: Portfile.2 added

Changed 12 years ago by neverpanic (Clemens Lang)

Attachment: config.log added

config.log from buildTree/

comment:10 Changed 12 years ago by neverpanic (Clemens Lang)

Keywords: lack-of-interest added
Resolution: invalid
Status: assignedclosed

Feel free to re-open.

Note: See TracTickets for help on using tickets.