Opened 8 years ago

Closed 8 years ago

#52121 closed update (fixed)

GNOME girl: version 9.5.1, please update MacPorts package

Reported by: ole@… Owned by: dbevans (David B. Evans)
Priority: Normal Milestone:
Component: ports Version: 2.3.4
Keywords: maintainer Cc:
Port: girl

Description

GNOME girl 9.5.1 is available.

Attachments (2)

Portfile (2.3 KB) - added by ole@… 8 years ago.
Portfile for GNOME girl 9.5.1
girl-9.5.1-revisions.diff (1000 bytes) - added by dbevans (David B. Evans) 8 years ago.
Proposed revisions to the submitted Portfile

Download all attachments as: .zip

Change History (9)

Changed 8 years ago by ole@…

Attachment: Portfile added

Portfile for GNOME girl 9.5.1

comment:1 Changed 8 years ago by dbevans (David B. Evans)

Keywords: maintainer added
Owner: changed from macports-tickets@… to devans@…
Status: newassigned

Ole, a few issues.

  • Please follow the MacPorts recommendations and re-submit your update as a unified patch against the existing Portfile. This helps us to see what changes are being made.
  • MacPorts policy is, in general, to only update to stable versions. For GNOME ports, this generally is indicated by version numbers with an odd minor version such as 9.5.1. To indicate that 9.5.1 is, in fact, a stable release, please change the livecheck type statement at the end of the Portfile from gnome to gnome-with-unstable.
  • Add the keywords haspath maintainer so that committers understand this immediately and give it a higher priority.
  • I've been following this port, so feel free to copy me on your posts to bring the ticket to my attention.

Thanks

Last edited 8 years ago by dbevans (David B. Evans) (previous) (diff)

comment:2 Changed 8 years ago by dbevans (David B. Evans)

Updates should be posted as unified patches using something like

diff -u Portfile.old Portfile.new

comment:3 Changed 8 years ago by dbevans (David B. Evans)

The Portfile you submitted fails to build for the following reasons:

  • checksums do not match the downloaded tarball
  • patchfile is outdated and should be removed

Attached is a patch releative to the Portfile you submitted that works for me including a fix for livecheck. Let me know if this is OK to commit or if you want to make other changes.

Changed 8 years ago by dbevans (David B. Evans)

Attachment: girl-9.5.1-revisions.diff added

Proposed revisions to the submitted Portfile

comment:4 Changed 8 years ago by dbevans (David B. Evans)

Type: defectupdate

comment:5 Changed 8 years ago by ole@…

Looks good, but I am not able to build girl. I get these msgs:

Error: The +llvm34 variant is not supported on Lion and later.

Error: cctools: Error executing llvm34: Invalid variant selected

Error: Unable to open port: Error evaluating variants

Error: Unable to execute port: upgrade totem failed

comment:6 in reply to:  5 Changed 8 years ago by dbevans (David B. Evans)

Replying to ole@…:

Looks good, but I am not able to build girl. I get these msgs:

Error: The +llvm34 variant is not supported on Lion and later.

Error: cctools: Error executing llvm34: Invalid variant selected

Error: Unable to open port: Error evaluating variants

Error: Unable to execute port: upgrade totem failed

As the errors say the variants for cctools have been recently updated and +llvm34 is no longer supported on Lion and later. See

port info cctools

for variants supported on your OS version and use one of these or just

sudo port install cctools

to install with the new latest default variant (+llvm38 on my machine running 10.10.5)

comment:7 Changed 8 years ago by dbevans (David B. Evans)

Resolution: fixed
Status: assignedclosed

Updated in r152332.

I'm assuming you got things worked out on your end with cctools. If not let us know. Thanks for the update.

Note: See TracTickets for help on using tickets.