pipc does not honour file name extension

[rvjansen]

Say we have two files, test.njp and test.pipe. When we run the command

pipc test.pipe

the file test.njp is still compiled; the test.pipe file is ignored, eventhough we specified its filename extension explicitly.

[rvjansen]

It is even worse with filename parsing here. When we have a test.njp file and a test.nrx already exists, the latter is rücksichtslos eradicated. Which is not in the best tradition of protecting people's work.

[rvjansen]

fixed in 802bb61 and ready for testing

[remesm]

Fix does not handle use case 3.4 Compiled pipeline from the command line (Pipelines Guide and Reference) correctly, and still removes an existing .nrx file.

$ echo "say 'hello'" >aplan.nrx
$ cat aplan.nrx 
say 'hello'
$ pipc '(aplan ) literal a man | cons'
( aplan  )  literal a man | cons
15=23312 0=-1 1722350893994>=1722350895994 0=0
$ ls aplan*
aplan.class

[rvjansen]

Ah - very well spotted. I only checked for the source .njp, not for the pipe name. René.On 30 Jul 2024, at 16:51, remesm ***@***.***> wrote: Fix does not handle use case 3.4 Compiled pipeline from the command line (Pipelines Guide and Reference) correctly, and still removes an existing .nrx file. $ echo "say 'hello'" >aplan.nrx $ cat aplan.nrx say 'hello' $ pipc '(aplan ) literal a man | cons' ( aplan ) literal a man | cons 15=23312 0=-1 1722350893994>=1722350895994 0=0 $ ls aplan* aplan.class —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were assigned.Message ID: ***@***.***>

[remesm]

fix [222e2b] when pipe name is specified between (), and also avoids overwriting any class.nrx specified as embedded netrexx code in the njp.
Please test

[rvjansen]

works very well, thanks!