Ticket #1429 (closed experimental: wontfix)

Opened 5 years ago

Last modified 3 years ago

Add -L support to parrot-nqp

Reported by: Austin_Hastings Owned by: pmichaud
Priority: normal Milestone:
Component: nqp Version: master
Severity: medium Keywords: cli
Cc: jkeenan Language:
Patch status: Platform: all

Description

The installed parrot-nqp binary does not support passing through various parrot flags, in particular -L for load searchpath.

Please add it.

(Note: this may be a general-purpose feature for all PCT systems)

Change History

  Changed 5 years ago by pmichaud

(general purpose feature...) Are you advocating that every Parrot language translator needs to support the -L option? If so, I think that's a non-starter.

I'll have to think a bit about whether I think it makes sense for parrot-nqp to support the -L option. It seems to me it may need to be a more generic Parrot feature than something that is specific to nqp or PCT.

Pm

follow-up: ↓ 4   Changed 5 years ago by coke

On Wed, Feb 10, 2010 at 11:16 AM, Parrot
<parrot-tickets@lists.parrot.org> wrote:
> #1429: Add -L support to parrot-nqp
> -----------------------------+----------------------------------------------
>  Reporter:  Austin_Hastings  |       Owner:  pmichaud
>     Type:  todo             |      Status:  new
>  Priority:  normal           |   Milestone:
> Component:  nqp              |     Version:  2.0.0
>  Severity:  medium           |    Keywords:
>     Lang:                   |       Patch:
>  Platform:                   |
> -----------------------------+----------------------------------------------
>
> Comment(by pmichaud):
>
>  (general purpose feature...)   Are you advocating that every Parrot
>  language translator needs to support the -L option?  If so, I think that's
>  a non-starter.
>
>  I'll have to think a bit about whether I think it makes sense for parrot-
>  nqp to support the -L option.  It seems to me it may need to be a more
>  generic Parrot feature than something that is specific to nqp or PCT.
>
>  Pm
>
> --
> Ticket URL: <https://trac.parrot.org/parrot/ticket/1429#comment:1>
> Parrot <https://trac.parrot.org/parrot/>
> Parrot Development
> _______________________________________________
> parrot-tickets mailing list
> parrot-tickets@lists.parrot.org
> http://lists.parrot.org/mailman/listinfo/parrot-tickets
>

It might be worth considering adding support to parrot for a PBCPATH
env var (something like java's CLASSPATH.)

-- 
Will "Coke" Coleda

  Changed 5 years ago by Austin_Hastings

Pmichaud:

My thought was that the PCT language all support --target= and --output= (along with some others) so adding support for -L (--bytecode-load-path=?) might make sense at the same level.

in reply to: ↑ 2   Changed 5 years ago by NotFound

Replying to coke:

It might be worth considering adding support to parrot for a PBCPATH env var (something like java's CLASSPATH.)

Added experimental feature in r44102, PARROT_LIBRARY and PARROT_INCLUDE env vars with the same effect as the -L and -I command line options.

  Changed 5 years ago by Austin_Hastings

I think I sense a PMC type in here. FileSearchPath with an RPA-like interface (indexed, plus PPSU ops) and a find_file method.

  Changed 5 years ago by coke

>  I think I sense a PMC type in here. `FileSearchPath` with an RPA-like
>  interface (indexed, plus PPSU ops) and a find_file method.

ParrotInterpreter is probably a perfectly good PMC to hang file search
information off of, if we're going that route.


-- 
Will "Coke" Coleda

  Changed 5 years ago by NotFound

The way used in r44102 was wrong, it only worked from parrot main executable, not from fakecutables or embedding. Changed in r45359

  Changed 5 years ago by fperrad

PARROT_LIBRARY and PARROT_INCLUDE env vars must support a separator for multi path.

This separator must be platform independent (no like Java's CLASSPATH which uses ';' on Windows and ':' on *nix)

  Changed 4 years ago by jkeenan

  • cc jkeenan added

Commenters: Can we get an update on the status of this ticket?

Thank you very much.

kid51

  Changed 4 years ago by whiteknight

  • keywords cli added
  • platform set to all
  • version changed from 2.0.0 to master
  • type changed from todo to experimental

I mentioned this issue yesterday at #parrotsketch, and the decision was to keep this item as experimental for now.

  Changed 3 years ago by pmichaud

  • status changed from new to closed
  • resolution set to wontfix

The new NQP (no -rx) and HLL::Compiler classes includes options for loading libraries at startup. If we still need to backport them into nqp-rx and PCT, file a new issue and perhaps someone can add them.

Thanks!

Pm

Note: See TracTickets for help on using tickets.