pbc_to_exe needs to be created as installable version aswell

[rg]

Right now the build version of pbc_to_exe gets installed. The attached patch creates an installable version just as for the other binaries.

[rg]

build installable_pbc_to_exe

[fperrad]

Patch doesn't work.

Fixed in r37072.

[rg]

Replying to fperrad:

Patch doesn't work.

Well sorry for that, it was supposed to be applied pre 0.9.1.

Fixed in r37072.

Sorry again, but no, you didn't fix it. Using a copy is not correct. That's the same as what's been installed in the first place, i.e. a pbc_to_exe binary that's linked against the shared libraries in the build directory.

You really need an installable version, built with --install. I'm attaching a new patch against r37077.

[rg]

new patch for pcb2exe

[fperrad]

currently (ie r37072) with a simple copy, I can do this :

$ mkdir somewhere
$ perl tools/dev/mk_language_shell.pl Xyz somewhere/xyz
$ cd somewhere/xyz
$ perl Configure.pl
$ make all

Now, if I apply your way :

$ parrot pbc_to_exe.pbc pbc_to_exe.pbc --install
$ make install-dev

building the executable xyz fails with :

pbc_to_exe xyz.pbc
Unable to open filehandle
current instr.: '_config' pc -1 ((unknown file):-1)
called from Sub '_config' pc -1 ((unknown file):-1)
called from Sub '_config' pc 6 (library/config.pir:54)
called from Sub 'handle_args' pc 248 (tools\dev\pbc_to_exe.pir:146)
called from Sub 'main' pc 68 (tools\dev\pbc_to_exe.pir:30)

[rg]

Replying to fperrad:

currently (ie r37072) with a simple copy, I can do this : $ mkdir somewhere
$ perl tools/dev/mk_language_shell.pl Xyz somewhere/xyz
$ cd somewhere/xyz
$ perl Configure.pl
$ make all

Unfortunately, you're in trouble if you (re)move your build directory before the make.

Now, if I apply your way : $ parrot pbc_to_exe.pbc pbc_to_exe.pbc --install
$ make install-dev building the executable xyz fails with : pbc_to_exe xyz.pbc
Unable to open filehandle

[...]

Good catch. My investigation turned up you found a second dependency on the build directory. This one is for config.fpmc. The problem is in lines 51-52 of runtime/parrot/library/config.pir (which gets included in pbc_to_exe). That path does not work with an installed layout.

Can anyone point me to an example of how to adjust that path for an installed parrot?

Or alternatively, does pbc_to_exe have to load config.fpmc at all? Couldn't it get the values from its own interpreter conf_hash?

I'll keep looking, but I'd appreciate some pointers.

[fperrad]

Since r37083, the error message contains the full path of config.fpmc.

We try to open @prefix@/runtime/parrot/include/config.fpmc.

And we couldn't build a path like @prefix@/lib/parrot/0.9.1-devel/include/config.fpmc, because we don't know at this time the version (it's a part of the content of config.fpmc).

A solution could be :

• use @prefix@/config.fpmc as path
• in build tree, build config.fpmc instead of runtime/parrot/include/config.fpmc, and not install it
• install install_config.fpmc as @prefix@/config.fpmc

We could use the same convention as executable and rename install_config.fpmc to installable_config.fpmc

[fperrad]

Fixed with r37090, r37091 & r37092.