Opened 5 years ago

Closed 4 years ago

#51497 closed defect (fixed)

unittest-cpp @1.4.1 Upgrade to version 1.6.1

Reported by: jmgc Owned by: ryandesign (Ryan Schmidt)
Priority: Normal Milestone:
Component: ports Version: 2.3.4
Keywords: Cc:
Port: unittest-cpp

Description

Since the last Portfile update, the UnitTest-C++ project has changed its host and address, so the port does not work anymore. A new Portfile has been prepared using the version 1.6.1. It has been cleaned as the installation has been simplified. A patch has been created with the changes.

Attachments (1)

Portfile-unittest-cpp.diff (1.9 KB) - added by jmgc 5 years ago.
Portfile patch

Download all attachments as: .zip

Change History (10)

comment:1 Changed 5 years ago by mf2k (Frank Schima)

Thanks. Some comments:

  • The revision line should be deleted because it starts at 0 when increasing the version and that is the default value.
  • Add a sha256 checksum and remove the obsolete md5 and sha1 checksums unless upstream provides them.
  • Since it is now hosted at Github, the github portgroup should be used. This will allow the Portfile to be simplified.

comment:2 Changed 5 years ago by jmgc

Sorry, where can I find the github portgroup documentation to use it properly? Thanks in advance

comment:3 Changed 5 years ago by mf2k (Frank Schima)

You have to look at the portgroup code or use another port as an example - e.g. elixir.

Last edited 5 years ago by mf2k (Frank Schima) (previous) (diff)

comment:4 Changed 5 years ago by jmgc

Sorry, but I am not able to request the file https://github.com/unittest-cpp/unittest-cpp/tarball/v1.6.1.tar.gz using the portgroup. I am quite sure that I am doing something wrong, but I do not know how to solve it. The present lines for github are:

PortGroup github 1.0

github.setup unittest-cpp unittest-cpp 1.6.1 v

Changed 5 years ago by jmgc

Attachment: Portfile-unittest-cpp.diff added

Portfile patch

comment:5 Changed 5 years ago by jmgc

The patch now uses the github port group.

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

You removed the muniversal portgroup. Does it build universal properly now in the new version?

comment:7 Changed 4 years ago by ryandesign (Ryan Schmidt)

Owner: changed from macports-tickets@… to ryandesign
Status: newaccepted

Has duplicate #52926.

comment:8 in reply to:  6 Changed 4 years ago by ryandesign (Ryan Schmidt)

Replying to mf2k:

You removed the muniversal portgroup. Does it build universal properly now in the new version?

Yes. The muniversal portgroup was used because the project did not use autoconf. The new version does use autoconf so muniversal is no longer needed.

comment:9 Changed 4 years ago by ryandesign (Ryan Schmidt)

Resolution: fixed
Status: acceptedclosed

In 5e85890c/macports-ports:

unittest-cpp: Update to 1.6.1

Closes: #51497

Note: See TracTickets for help on using tickets.