Opened 13 years ago
Closed 13 years ago
#29179 closed submission (fixed)
palmkit @ 1.0.32
Reported by: | humem (humem) | Owned by: | ryandesign (Ryan Carsten 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)
Change History (12)
Changed 13 years ago by humem (humem)
comment:1 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)
Owner: | changed from macports-tickets@… to ryandesign@… |
---|---|
Status: | new → assigned |
comment:2 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
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 13 years ago by ryandesign (Ryan Carsten 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 13 years ago by humem (humem)
Attachment: | Portfile.2 added |
---|
Changed 13 years ago by humem (humem)
Attachment: | patch-scripts.diff added |
---|
comment:4 Changed 13 years ago by humem (humem)
Resolution: | fixed |
---|---|
Status: | closed → reopened |
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 13 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
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 13 years ago by humem (humem)
Attachment: | Portfile.diff added |
---|
a patchfile for incorrect tmpdir
comment:6 Changed 13 years ago by humem (humem)
Resolution: | fixed |
---|---|
Status: | closed → reopened |
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 13 years ago by humem (humem)
Attachment: | patch-src.diff added |
---|
a patchfile for incorrect tmpdir
comment:7 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Thanks, r78149.
Thanks.