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.

Attachment: zoxide-0.9.4.tar.gz
Description: application/tar-gz

Reply via email to