Hi Kurt,

thanks for testing and enhancements, have them locally added to my tree,
waiting for unlock.

cheers,
Sebastian

On Sun, Oct 4, 2026 at 10:11 PM Kurt Miller <[email protected]>
wrote:

> 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>
>
>

-- 
https://buzzdeee.reitenba.ch

Reply via email to