On Oct 3, 2026, at 2:04 PM, Sebastian Reitenbach 
<[email protected]> wrote:
> 
> Hi,
> 
> thanks for the feedback. Indeed, the shared lib is embedded, and seems 
> sufficient to make Autopsy happy.
> All other items mentioned updated as well. For some reason, make update-plist 
> didn't change the amd64 -> ${ARCH},
> but manually changing it, it at least doesn't revert it.
> 
> cheers,
> Sebastian
> 

Looking good.

I tested it on the 4 supported arch and noticed that MACHINE_ARCH
instead of ARCH was needed for aarch64 and on sparc64 the jar arch
does not match either ARCH or MACHINE_ARCH. So a custom ARCH is needed
as follows:

JAR_ARCH =      ${MACHINE_ARCH:S/sparc64/sparcv9/}

SUBST_VARS+=    JAR_ARCH

with do-install and PLIST using ${JAR_ARCH}. 

With that final tweak, ok kurt@ after unlock.

> 
> On Sat, Oct 3, 2026 at 2:02 AM Kurt Miller <[email protected]> 
> wrote:
> On Oct 2, 2026, at 3:27 PM, Sebastian Reitenbach 
> <[email protected]> wrote:
> > 
> > This is the cleaned-up, updated version, installing the .jars in 
> > $MODJAVA_JAR_DIR
> > It's needed dependency for Autopsy.
> > 
> > obviously, for after unlock.
> > 
> > any testing, feedback etc. welcome.
> > 
> 
> I took a brief look at the Makefile and have some suggestions. I
> did not build or test this yet.
> 
> Yes this should go under archivers. We try to avoid putting ports
> in the java category as the primary category. 
> 
> -CATEGORIES =   archivers java
> +CATEGORIES =   archivers
> 
> java.port.mk will add the secondary java category for you.
> 
> -MODJAVA_VER =  21
> +MODJAVA_VER =  11+
> 
> According to the build instructions this port is Java JDK 1.5
> or higher. We set the lowest possible jdk for MODJAVA_VER and
> generally use ‘+' to allow higher versions to use it as well. Right
> now our lowest jdk is 11 and ‘+' make sense unless a higher jdk
> can’t use the jars.
> 
>         # Install native JNI shared object
>         ${INSTALL_DATA_DIR} ${PREFIX}/lib/sevenzipjbinding
>         ${INSTALL_DATA} ${WRKBUILD}/jbinding-cpp/lib7-Zip-JBinding.so \
>                 ${PREFIX}/lib/sevenzipjbinding/
> 
> Is this really needed? Usually jni shared objects are stored within the
> jar and java can utilize it just fine there. Please show what fails
> if you do not install this in the lib subdirectory.
> 
> Also something is not right with ARCH and your PLIST:
> 
> share/java/classes/sevenzipjbinding-OpenBSD-amd64.jar
> 
> That amd64 should not be in your PLIST. I thought make update-plist
> would substitute that for you back to ${ARCH}.
> 
> Now looking back at the Makefile and:
> 
> -SUBST_VARS = ARCH
> 
> is not needed because bsd.port.mk already has ARCH in SUBST_VARS.
> 
> Best,
> -Kurt
> 
> > cheers,
> > Sebastian
> > 
> > On Sat, Sep 19, 2026 at 10:07 AM Sebastian Reitenbach 
> > <[email protected]> wrote:
> > Hi,
> > 
> > this could also go under archivers, if preferred.
> > 
> > 7-Zip-JBinding is a free cross-platform java binding of 7-Zip free
> > compress/decompress library.
> > 
> > this is the fork from sleuthkit, as they have some specific patches, that 
> > sleuthkit/autopsy needs.
> > 
> > testing, feedback of any sort welcome.
> > 
> > cheers,
> > Sebastian
> > 
> > -- 
> > https://buzzdeee.reitenba.ch
> > 
> > 
> > -- 
> > https://buzzdeee.reitenba.ch<sevenzipjbinding.tar.gz>
> 
> 
> 
> 
> -- 
> https://buzzdeee.reitenba.ch
> <sevenzipjbinding.tar.gz>

Reply via email to