Opened 10 years ago

Closed 10 years ago

#29179 closed submission (fixed)

palmkit @ 1.0.32

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

Description

This is a Portfile for palmkit, Publicly Available Language Modeling toolKIT.

Attachments (5)

Portfile (1.9 KB) - added by humem (humem) 10 years ago.
Portfile.2 (2.0 KB) - added by humem (humem) 10 years ago.
patch-scripts.diff (872 bytes) - added by humem (humem) 10 years ago.
Portfile.diff (617 bytes) - added by humem (humem) 10 years ago.
a patchfile for incorrect tmpdir
patch-src.diff (1.9 KB) - added by humem (humem) 10 years ago.
a patchfile for incorrect tmpdir

Download all attachments as: .zip

Change History (12)

Changed 10 years ago by humem (humem)

Attachment: Portfile added

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

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

Thanks.

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

Resolution: fixed
Status: assignedclosed

Added palmkit in r77997 with these changes:

  • changed a few lone tabs to spaces
  • changed zlib from a build dependency to a library dependency (you can see with otool -L that things installed by this port do link with libz.dylib)

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

In r77999 I added support for build_arch. Unfortunately, when trying to build i386 on Snow Leopard, I get a build error. I also got a build error when trying to build universal i386 x86_64 so I didn't enable the universal variant.

Changed 10 years ago by humem (humem)

Attachment: Portfile.2 added

Changed 10 years ago by humem (humem)

Attachment: patch-scripts.diff added

comment:4 Changed 10 years ago by humem (humem)

Resolution: fixed
Status: closedreopened

I should have been careful before I submitted the port, but I found that enabling zlib makes an executable cause a bus error and paths specified in script files are not correct. Could you please replace Portfile with Portfile.2 and append patch-scripts.diff? Thanks in advance.

comment:5 Changed 10 years ago by ryandesign (Ryan Schmidt)

Resolution: fixed
Status: reopenedclosed

Sure: r78022.

When updating a port, it's better to submit a diff of the portfile, rather than a complete new portfile; this makes it easier to evaluate your changes.

Changed 10 years ago by humem (humem)

Attachment: Portfile.diff added

a patchfile for incorrect tmpdir

comment:6 Changed 10 years ago by humem (humem)

Resolution: fixed
Status: closedreopened

I added Portfile.diff and patch-src.diff to fix wrong tmpdir specified in the src directory. Some executables failed against large input data.

Changed 10 years ago by humem (humem)

Attachment: patch-src.diff added

a patchfile for incorrect tmpdir

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

Resolution: fixed
Status: reopenedclosed

Thanks, r78149.

Note: See TracTickets for help on using tickets.