Opened 9 years ago

Closed 9 years ago

#46369 closed submission (fixed)

port submission: libetonyek

Reported by: RJVB (René Bertin) Owned by: mkae (Marko Käning)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: ryandesign (Ryan Carsten Schmidt)
Port: libetonyek

Description

This is a proposal for port:libetonyek, a library for parsing Apple Keynote documents and one of Calligra's dependencies.

Attachments (1)

Portfile​ (1.7 KB) - added by RJVB (René Bertin) 9 years ago.
obfuscated maintainer address

Download all attachments as: .zip

Change History (14)

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

The license field should be MPL-2 (not MPL 2.0).

comment:2 Changed 9 years ago by mkae (Marko Käning)

Cc: mk@… added

Cc Me!

comment:3 Changed 9 years ago by mkae (Marko Käning)

Cc: mk@… removed
Owner: changed from macports-tickets@… to mk@…
Version: 2.3.3

comment:4 Changed 9 years ago by mkae (Marko Käning)

Resolution: → fixed
Status: new → closed

Committed in r137817.

Thanks, René!

comment:5 Changed 9 years ago by mkae (Marko Käning)

René, Ryan pointed out on the dev-ML that cppunit might not be needed at runtime. Do you agree?

comment:6 Changed 9 years ago by mkae (Marko Käning)

Resolution: fixed
Status: closed → reopened

comment:7 Changed 9 years ago by mkae (Marko Käning)

Cc: ryandesign@… added

@Ryan: I am also not sure whether I should rev-bump this port in case I remove the dependency on cppunit.

comment:8 Changed 9 years ago by RJVB (René Bertin)

I suppose Ryan is right, but to be honest my guess is as good as yours. Is there a depends_ variant for testing dependencies?

comment:9 in reply to:  8 Changed 9 years ago by larryv (Lawrence Velázquez)

There is not, until someone deals with #38208.

comment:10 Changed 9 years ago by mkae (Marko Käning)

This is what happens if I try to build the lib without cppunit: 

:info:configure checking for CPPUNIT... no
:info:configure configure: error: Package requirements (cppunit) were not met:
:info:configure
:info:configure No package 'cppunit' found
:info:configure
:info:configure Consider adjusting the PKG_CONFIG_PATH environment variable if you
:info:configure installed software in a non-standard prefix.
:info:configure
:info:configure Alternatively, you may set the environment variables CPPUNIT_CFLAGS
:info:configure and CPPUNIT_LIBS to avoid the need to call pkg-config.
:info:configure See the pkg-config man page for more details.

Changed 9 years ago by RJVB (René Bertin)

Attachment: Portfile​ added

obfuscated maintainer address

comment:11 Changed 9 years ago by RJVB (René Bertin)

that's probably how I found out about the cppunit dependency.

Try the refreshed Portfile. It updates the port to v0.1.2, and avoids building the tests. I've also disabled a number of warnings that seem both spurious and are extremely chatty (if the project really required C99 and/or C++11, it w/should set the corresponding compiler flags). Suppressing the warnings speeds up the build with some 14% for me (and makes it a lot easier to find real errors).

comment:12 Changed 9 years ago by mkae (Marko Käning)

Done in r137998. Thanks!

comment:13 Changed 9 years ago by mkae (Marko Käning)

Resolution: → fixed
Status: reopened → closed
Note: See TracTickets for help on using tickets.