Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#29161 closed submission (fixed)

tinysvm @ 0.09 - new port

Reported by: humem (humem) Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: science Cc:
Port: tinysvm

Description

This is a Portfile for TinySVM, an implementation of SVMs.

Attachments (2)

Portfile​ (1.4 KB) - added by humem (humem) 13 years ago.
Portfile.2​ (1.6 KB) - added by humem (humem) 13 years ago.

Download all attachments as: .zip

Change History (6)

Changed 13 years ago by humem (humem)

Attachment: Portfile​ added

Changed 13 years ago by humem (humem)

Attachment: Portfile.2​ added

comment:1 Changed 13 years ago by humem (humem)

Portfile.2 enables a universal build and make check.

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

Owner: changed from macports-tickets@… to ryandesign@…
Status: new → assigned

Thanks.

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

Resolution: → fixed
Status: assigned → closed

Added in r77996 with these changes:

  • You had the port name as "tinysvm" and had set distfiles because the distfile had "TinySVM" in the name, but you didn't set worksrcdir, meaning the port would have failed on case-sensitive filesystems. I was going to suggest setting distname instead, since that would take care of both distfiles and worksrcdir, but in fact, it's simplest to just set the port name to "TinySVM"; the only reason we didn't used to do that is that years ago, MacPorts used to have a bug when you specified the port name on the command line with a different case than the port specified. But this bug was fixed a long time ago.
  • did not add your universal stuff since the port did not need it; it has a standard autoconf-based configure script and so it works fine with the default universal variant already provided in MacPorts base -- except that the configure script deliberately overwrites our CFLAGS and CXXFLAGS; I added a patch so it wouldn't do that anymore
  • removed "use_autoreconf yes"; it seems to build fine with the existing configure script, even if it is generated by an old autoconf
  • changed "configure.env-append CPPFLAGS=-D__GNU_LIBRARY__" to "configure.cppflags-append -D__GNU_LIBRARY__"
  • changed master_sites to avoid double slash in download URL
  • changed livecheck.url using lindex, just in case a second download URL ever gets added to the portfile
  • indicated the version of the license in the license field
  • changed "--enable-shared=no" to "--disable-shared" which seems more straightforward

I assume the reason you're disabling the shared library is that it did not build? It didn't build for me when I tried it. Otherwise we would prefer to have the shared library.

comment:4 in reply to:  3 Changed 13 years ago by humem (humem)

Replying to ryandesign@…:

Thank you for your careful support!

I assume the reason you're disabling the shared library is that it did not build? It didn't build for me when I tried it. Otherwise we would prefer to have the shared library.

Exactly. I could not make it with the shared library.

Note: See TracTickets for help on using tickets.