Thank you aisha, for testing the port.
Do I need to resend the fixed port, or the one you have sent will do?



On Tue, Sep 10, 2024 at 8:30 AM aisha <openbsd.po...@aisha.cc> wrote:

> On 24/09/09 10:26PM, Neel chakraborty wrote:
> > Hello everyone,
> >
> > I am happy to announce that I have successfully ported Zoxide
> > (https://github.com/ajeetdsouza/zoxide) to OpenBSD. Zoxide is a modern
> > replacement for the cd command, written in Rust.
> >
> > This is my first port, and I would greatly appreciate it if you could
> > test it and let me know if there are any issues. I am ready to address
> > any problems that arise as soon as possible.
> >
> > I have included the gzipped tarball of the port directory for your
> > review.
> >
> > Best regards,
> > Neel Chakraborty
>
>
> Good start on the port, mostly correct with nits in formatting and some
> comments on the Makefile, which I've inlined with fixes
>
> > Everything should be aligned on the right of the "=" as much as possible
>
> COMMENT =               zoxide is a smarter cd command
>
> > comment should not end with "."
>
> DIST_TUPLE =            github ajeetdsouza zoxide v0.9.4 .
>
> CATEGORIES =            sysutils
>
> MAINTAINER =            Neel Chakraborty <neelroboinfo...@gmail.com>
>
> # MIT
> PERMIT_PACKAGE =        Yes
>
> > PERMIT_PACKAGE should have license on top of it
>
> WANTLIB =               ${MODCARGO_WANTLIB}
>
> MODULES =               devel/cargo
>
> SEPARATE_BUILD =        Yes
>
> CONFIGURE_STYLE =       cargo
>
> > didn't need the INTERACTIVE set to NO as it's the default
>
> post-install:
>         ${INSTALL_MAN} ${WRKSRC}/man/man1/zoxide.1 ${PREFIX}/man/man1/
>         ${INSTALL_MAN} ${WRKSRC}/man/man1/zoxide-add.1 ${PREFIX}/man/man1/
>         ${INSTALL_MAN} ${WRKSRC}/man/man1/zoxide-import.1
> ${PREFIX}/man/man1/
>         ${INSTALL_MAN} ${WRKSRC}/man/man1/zoxide-init.1 ${PREFIX}/man/man1/
>         ${INSTALL_MAN} ${WRKSRC}/man/man1/zoxide-query.1
> ${PREFIX}/man/man1/
>         ${INSTALL_MAN} ${WRKSRC}/man/man1/zoxide-remove.1
> ${PREFIX}/man/man1/
>
>
> .include "crates.inc"
>
> .include <bsd.port.mk>
>
>
>
> Not 100% sure how much I like this command...
>
> > obsd-ports# zoxide init ksh
> > error: invalid value 'ksh' for '<SHELL>'
> >   [possible values: bash, elvish, fish, nushell, posix, powershell,
> xonsh, zsh]
> >
> >   tip: a similar value exists: 'zsh'
> >
> > For more information, try '--help'.
>
>
> fixed port attached.
>
> OK aisha if someone wants to import this.
>

Reply via email to