Ticket #1429 (closed experimental: wontfix)

Opened 10 years ago

Last modified 8 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 10 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 10 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 10 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 10 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 10 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 10 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 10 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 10 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 10 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 9 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 8 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.