Opened 13 years ago
Closed 13 years ago
#29787 closed submission (fixed)
apache solr @ 3.2.0 - newport
Reported by: | hayato_1980@… | Owned by: | ryandesign (Ryan Carsten Schmidt) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 1.9.2 |
Keywords: | java search | Cc: | |
Port: | apache-solr |
Description
This is Portfile Apache Solr, search platform based on Apache Lucene, java.
Attachments (2)
Change History (4)
Changed 13 years ago by hayato_1980@…
Changed 13 years ago by hayato_1980@…
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 |
Committed in r79407 with some changes:
distfiles ${name}-${version}.tgz
This line is unnecessary.
checksums sha1 78caf37494603d69096709ed31455e6e433bff63
We want to use more than one checksum type. For now, we suggest sha1 and rmd160.
set extracted ${workpath}/${name}-${version}
This value is already available in the variable ${worksrcpath}.
set target ${destroot}/${prefix}/share/java/${name}-${version}
There should be no "/" before ${prefix}. ${prefix} already begins with a "/".
fs-traverse -tails file ${extracted}/example {
switch -exact [ file type ${extracted}/example/${file}] {
directory {
xinstall -d -m 755 ${target}/${file}
}
file {
xinstall -m 644 ${extracted}/example/${file} ${target}/${file}
}
}
}
Why not simply copy the entire directory?
xinstall -m 755 ${filespath}/solr ${prefix}/bin
This needs to be installed into ${destroot}${prefix}/bin so that it is registered to the port and removed when the port is uninstalled.
This script assumes prefix is /opt/local but it might not be. Use a "@PREFIX@" placeholder in the script, then reinplace that in the portfile.
JAVA_HOME is also hardcoded into this script, and could be reinplaced from the portfile.
The variable CMDNAME in this script was never used. I assume is was supposed to have been used in the usage message.
post-deactivate {}
post-activate {}
These lines are unnecessary.
Thanks.