vcbuild small fix for plperl

Started by Magnus Haganderalmost 20 years ago4 messagespatches
Jump to latest
#1Magnus Hagander
magnus@hagander.net

Typo in the changes to plperl - uses wrong dir, and had a missing slash.

Also fixes error checking for xsubpp - it was broken in a way that hid
the problem above when run more than once (which is the normal case when
developing).

//Magnus

Attachments:

vcbuild_plperl.diffapplication/octet-stream; name=vcbuild_plperl.diffDownload+7-7
#2Andrew Dunstan
andrew@dunslane.net
In reply to: Magnus Hagander (#1)
Re: vcbuild small fix for plperl

Magnus Hagander wrote:

Typo in the changes to plperl - uses wrong dir, and had a missing slash.

Also fixes error checking for xsubpp - it was broken in a way that hid
the problem above when run more than once (which is the normal case when
developing).

I have just noticed that this file appears to have the perl version
hardcoded (perl58.lib). That doesn't seem sensible ... can it be avoided?

cheers

andrew

#3Neil Conway
neilc@samurai.com
In reply to: Magnus Hagander (#1)
Re: vcbuild small fix for plperl

On Tue, 2006-11-21 at 18:45 +0100, Magnus Hagander wrote:

Typo in the changes to plperl - uses wrong dir, and had a missing slash.

Applied, thanks.

-Neil

#4Magnus Hagander
magnus@hagander.net
In reply to: Andrew Dunstan (#2)
Re: vcbuild small fix for plperl

Typo in the changes to plperl - uses wrong dir, and had a

missing slash.

Also fixes error checking for xsubpp - it was broken in a

way that hid

the problem above when run more than once (which is the normal case
when developing).

I have just noticed that this file appears to have the perl
version hardcoded (perl58.lib). That doesn't seem sensible
... can it be avoided?

It's definitly on my long-term TODO.

//Magnus