Re: [UPDATE] math/lrs -> 071a

2020-11-21 Thread Dimitri Karamazov
On Sat, November 21, 2020 13:05, Stuart Henderson wrote: > Now it's committed here's some clean-up. > > > - don't patch away upstream's INCLUDEDIR/LIBDIR mechanism to replace > it with something using CPPFLAGS/LDFLAGS which does exactly the same; > simplifies the patch > > patch-makefile | 115

Re: [UPDATE] math/lrs -> 071a

2020-11-21 Thread Stuart Henderson
On 2020/11/21 14:08, Dimitri Karamazov wrote: > On Sat, November 21, 2020 13:05, Stuart Henderson wrote: > > Now it's committed here's some clean-up. > > > > > > - don't patch away upstream's INCLUDEDIR/LIBDIR mechanism to replace > > it with something using CPPFLAGS/LDFLAGS which does exactly the

Re: [UPDATE] math/lrs -> 071a

2020-11-21 Thread Stuart Henderson
Now it's committed here's some clean-up. - don't patch away upstream's INCLUDEDIR/LIBDIR mechanism to replace it with something using CPPFLAGS/LDFLAGS which does exactly the same; simplifies the patch patch-makefile | 115 +++--- 1 file changed, 23

Re: [UPDATE] math/lrs -> 071a

2020-11-18 Thread Dimitri Karamazov
On Wed, November 18, 2020 22:02, Stuart Henderson wrote: > On 2020/11/18 21:14, Edd Barrett wrote: > >> On Wed, Nov 18, 2020 at 10:42:17AM -, Dimitri Karamazov wrote: >> >>> diff attached below with all the rest of the changes. >> >> Thanks! >> >> >> I was about to commit this when I noticed

Re: [UPDATE] math/lrs -> 071a

2020-11-18 Thread Stuart Henderson
On 2020/11/18 21:14, Edd Barrett wrote: > On Wed, Nov 18, 2020 at 10:42:17AM -, Dimitri Karamazov wrote: > > diff attached below with all the rest of the changes. > > Thanks! > > I was about to commit this when I noticed that there's a sole compiler > invocation which is not honouring our

Re: [UPDATE] math/lrs -> 071a

2020-11-18 Thread Edd Barrett
On Wed, Nov 18, 2020 at 10:42:17AM -, Dimitri Karamazov wrote: > diff attached below with all the rest of the changes. Thanks! I was about to commit this when I noticed that there's a sole compiler invocation which is not honouring our flags: ``` cc -DGMP -DMA lrsnash.c lrsnashlib.c

Re: [UPDATE] math/lrs -> 071a

2020-11-18 Thread Dimitri Karamazov
On Wed, November 18, 2020 11:14, Stuart Henderson wrote: > On 2020/11/18 02:39, Dimitri Karamazov wrote: > >> On Tue, November 17, 2020 21:01, Edd Barrett wrote: >> >>> If another dev OKs this (and specifically checks the 'a' suffix on the >>> version number), I'd be happy to commit it. >>> > >

Re: [UPDATE] math/lrs -> 071a

2020-11-18 Thread Stuart Henderson
On 2020/11/18 02:39, Dimitri Karamazov wrote: > On Tue, November 17, 2020 21:01, Edd Barrett wrote: > > I think the comment should say 1.0, since before you patched it the > > makefile was like: > > > Attached below with given changes > > > > > You might want to try to feed parts of that makefile

Re: [UPDATE] math/lrs -> 071a

2020-11-17 Thread Dimitri Karamazov
On Tue, November 17, 2020 21:01, Edd Barrett wrote: > I think the comment should say 1.0, since before you patched it the > makefile was like: > Attached below with given changes > > You might want to try to feed parts of that makefile patch upstream btw. > Will do > > If another dev OKs this

Re: [UPDATE] math/lrs -> 071a

2020-11-17 Thread Edd Barrett
Hi, On Mon, Nov 16, 2020 at 10:58:35AM -, Dimitri Karamazov wrote: > 'make update' works just fine here, I don't what happened in your case. Well, I tested using `pkg_add -u` and it worked. > +SHARED_LIBS =lrs 0.0 # 0.0 I think the comment should say 1.0, since

Re: [UPDATE] math/lrs -> 071a

2020-11-16 Thread Dimitri Karamazov
On Mon, November 16, 2020 10:01, Edd Barrett wrote: > Hi, > > > On Mon, Nov 16, 2020 at 04:45:49AM -, Dimitri Karamazov wrote: > >> +V = 071a >> +PKGNAME = lrs-${V}lpha >> > I should've tested because making changes. 'make update' works just fine here, I don't what

Re: [UPDATE] math/lrs -> 071a

2020-11-16 Thread Edd Barrett
Hi, On Mon, Nov 16, 2020 at 04:45:49AM -, Dimitri Karamazov wrote: > +V = 071a > +PKGNAME =lrs-${V}lpha That's not right. Looking at the releases directory, 071a came *after* 071, so it's not an alpha as far as I can see:

Re: [UPDATE] math/lrs -> 071a

2020-11-16 Thread Dimitri Karamazov
On Sun, November 15, 2020 21:58, Edd Barrett wrote: > Hi, > > > Thanks for looking at this. > > > On Sun, Nov 15, 2020 at 05:38:53PM -, Dimitri Karamazov wrote: > >> Any comments/OK's? >> > > This looks good to me. > > > I'm a little confused as to why `make update` doesn't work, but `make >

Re: [UPDATE] math/lrs -> 071a

2020-11-15 Thread Edd Barrett
Hi, Thanks for looking at this. On Sun, Nov 15, 2020 at 05:38:53PM -, Dimitri Karamazov wrote: > Any comments/OK's? This looks good to me. I'm a little confused as to why `make update` doesn't work, but `make install` does an update: ``` $ make update $ make install ===> lrs-071a depends

[UPDATE] math/lrs -> 071a

2020-11-15 Thread Dimitri Karamazov
I made a port for lrslib didn't know it already existed under a different guise. Let me know if I'm missing something in here. Your tests are successful Update lrs 042 to 071a Version 7.1 is a major revision completing the move to an all C library begun in 7.0 which was work in progress and has