Re: [NEW] devel/pystring: required for opencolorio, blender update

2022-03-02 Thread Dimitri Karamazov
On Wed, Mar 02, 2022 at 11:27:12AM +0100, Omar Polo wrote:
> Dimitri Karamazov  wrote:
> > On Tue, Mar 01, 2022 at 10:05:09PM -0500, Daniel Dickman wrote:
> > > I think it needs
> > > 
> > > WANTLIB += ${COMPILER_LIBCXX} m
> > > 
> > > with that added, ok daniel@ for someone to import
> > > 
> 
> Diff against your makefile and tarball attached, if it's fine for you
> it's ok op@ for someone to import :)
I've reattached the tarball with the 'build' log removed.

I appreciate those changes, hope some one import this quick.
This port is prerequisite for the opencolorio-2.1.1/blender-3.0.1 update.

regards,
  Dimitri


pystring.tar.gz
Description: Binary data


Re: [NEW] devel/pystring: required for opencolorio, blender update

2022-03-02 Thread Omar Polo
Dimitri Karamazov  wrote:
> On Tue, Mar 01, 2022 at 10:05:09PM -0500, Daniel Dickman wrote:
> > I think it needs
> > 
> > WANTLIB += ${COMPILER_LIBCXX} m
> > 
> > with that added, ok daniel@ for someone to import
> > 
> > On Tue, Mar 1, 2022 at 9:42 PM Dimitri Karamazov
> >  wrote:
> > >
> > > Information for inst:pystring-1.1.3
> > >
> > > Comment:
> > > provides C++ functions analgous to python string methods
> > >
> > > Required by:
> > > blender-3.0.1
> > > opencolorio-2.1.1v0
> > >
> > > Description:
> > > This port provides python string methods in the form of C++
> > > functions with similar interface and behaviour.
> > >
> > > Maintainer: Dimitri Karamazov 
> > >
> > > WWW: https://github.com/imageworks/pystring
> Done.
> 
> regards,
>   Dimitri

Portcheck complains that with c++ libs in WANTLIB the default COMPILER
should be tweaked too.

You could also avoid patching the makefile by using MAKE_FLAGS and
FAKE_FLAGS, and with a little post-install target to install the header.

Diff against your makefile and tarball attached, if it's fine for you
it's ok op@ for someone to import :)

(while there i've also tweaked COMMENT and DESCR, feel free to discard
those changes)

If I've understood correctly, we're also in the process of dropping the
RCS tag, so i've removed them from the makefile and update-plist removed
it from the PLIST.

--- Makefile.orig   Wed Mar  2 11:22:35 2022
+++ MakefileWed Mar  2 11:22:27 2022
@@ -1,7 +1,5 @@
-# $OpenBSD: Makefile.template,v 1.91 2021/11/09 13:43:15 kn Exp $
+COMMENT =  C++ functions analgous to python string methods
 
-COMMENT =  provides C++ functions analgous to python string methods
-
 GH_ACCOUNT =   imageworks
 GH_PROJECT =   pystring
 GH_TAGNAME =   v1.1.3
@@ -17,8 +15,17 @@
 
 WANTLIB += ${COMPILER_LIBCXX} m
 
-MAKE_ENV = CXX=${CXX}
+COMPILER = base-clang ports-gcc
 
+MAKE_FLAGS =   CXX=${CXX} \
+   CXXFLAGS="${CXXFLAGS}"
+
+FAKE_FLAGS =   LIBDIR="${DESTDIR}${PREFIX}/lib"
+
 USE_GMAKE =Yes
+
+post-install:
+   ${INSTALL_DATA_DIR} ${PREFIX}/include/pystring
+   ${INSTALL_DATA} ${WRKSRC}/pystring.h ${PREFIX}/include/pystring
 
 .include 




pystring.tar.gz
Description: GNU Zip compressed data


Re: [NEW] devel/pystring: required for opencolorio, blender update

2022-03-01 Thread Daniel Dickman
I think it needs

WANTLIB += ${COMPILER_LIBCXX} m

with that added, ok daniel@ for someone to import

On Tue, Mar 1, 2022 at 9:42 PM Dimitri Karamazov
 wrote:
>
> Information for inst:pystring-1.1.3
>
> Comment:
> provides C++ functions analgous to python string methods
>
> Required by:
> blender-3.0.1
> opencolorio-2.1.1v0
>
> Description:
> This port provides python string methods in the form of C++
> functions with similar interface and behaviour.
>
> Maintainer: Dimitri Karamazov 
>
> WWW: https://github.com/imageworks/pystring