Opened 10 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)
Change History (14)
comment:1 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
comment:3 Changed 10 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 follow-up: 9 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 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.
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:13 Changed 9 years ago by mkae (Marko Käning)
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
The license field should be
MPL-2
(notMPL 2.0
).