On Wed, 28 Apr 2021, Tracey Emery wrote:

> On Wed, Apr 28, 2021 at 01:34:25PM -0600, Tracey Emery wrote:
> > On Wed, Apr 28, 2021 at 08:41:15PM +0200, Paco Esteban wrote:
> > > On Wed, 28 Apr 2021, Tracey Emery wrote:
> > > 
> > > > On Tue, Apr 27, 2021 at 06:59:40PM +0200, Paco Esteban wrote:
> > > > > Hi ports@
> > > > > 
> > > > > This is a new port of tree-sitter 
> > > > > (https://tree-sitter.github.io/tree-sitter/).
> > > > > From their site:
> > > > > 
> > > > >     Tree-sitter is a parser generator tool and an incremental parsing
> > > > >     library. It can build a concrete syntax tree for a source file and
> > > > >     efficiently update the syntax tree as the source file is edited.
> > > > > 
> > > > > This port will be needed when neovim 0.5 goes out (I'm testing nightly
> > > > > builds at the moment).
> > > > > 
> > > > > It's a bit odd because it has a C library and a Rust binary.
> > > > > Some tests fail, but it builds and neovim nightly takes it and builds
> > > > > and runs too (I have a problem with port-lib-depends-check, but that's
> > > > > a topic for another thread).
> > > > > 
> > > > > Any suggestions on my "franken-port" are very welcome.
> > > > > 
> > > > > ok to import ?
> > > > > 
> > > > > -- 
> > > > > Paco Esteban.
> > > > > 0x5818130B8A6DBC03
> > > > 
> > > > I must have missed a ulimit bump somewhere along the lines? Hmmm.
> > > > 
> > > > ===>  Compiler link: clang -> /usr/bin/clang
> > > > ===>  Compiler link: clang++ -> /usr/bin/clang++
> > > > ===>  Compiler link: cc -> /usr/bin/cc
> > > > ===>  Compiler link: c++ -> /usr/bin/c++
> > > > ===>  Generating configure for rust-1.50.0
> > > > ===>  Configuring for rust-1.50.0
> > > > Using /usr/ports/pobj/rust-1.50.0/config.site (generated)
> > > > datasize limit is too low - amd64 build takes approx 4GB
> > > 
> > > Added WANTLIB as per tracey's suggestion.
> > > Port attached for convenience.
> > > 
> > > Any flying oks ?
> > > 
> > > Cheers,
> > > Paco.
> > > 
> > > -- 
> > > Paco Esteban.
> > > 0x5818130B8A6DBC03
> > 
> > Not building on sparc64.
> > 
> > -- 
> > 
> > Tracey Emery
> 
> Paco, here is a newer version fixed for sparc64. It rearranges to get
> closer to Makefile.template ordering and gets rid of your patch. I'll
> give you a flying ok with this one as long as a go/rust person agrees
> that it's "ok." ;)
> 
> Then again, no documentation on where the hell MODCARGO_CRATES goes
> makes me sad. But, just trying to be a bit pedantic, since lack thereof
> in the past has bitten me. :)
> 
> I've tested this on sparc64 and amd64. Both build and appear to run.
> 
> What do you think about this version?

Looks good to me with a comment.  Why remove the @comment on PLIST ?
I've been told that only this line is correct:
@lib lib/libtree-sitter.so.${LIBtree-sitter_VERSION}

And they are symlinks anyway so ...

Cheers,

-- 
Paco Esteban.
0x5818130B8A6DBC03

Reply via email to