Ticket #63 (new defect)

Opened 8 months ago

Last modified 7 months ago

Shebang in marshall-py.py incompatible with mingw

Reported by: proegssilb@… Owned by:
Priority: major Milestone:
Component: build system Version:
Keywords: Cc:

Description

The shebang in CEL's bin/marshall-py.py is set to:

#!/usr/bin/python

On my installation of Mingw/Msys, installed April 7/8, Python is at /c/python25/python. The script fails to run and returns a generic path-not-found error. To work properly on my machine, the shebang needs to be set to:

#!/bin/env python

I have not tested this on any other machine, a real Linux installation, or Cygwin.

Attachments

Change History

  Changed 8 months ago by sunshine

This issue has manifested previously: ML-main-thread:1829

The original poster reported that he had resolved the problem but never explained how the resolution was made.

As noted in the mailing list thread, mk/jam/pymarshal.jam is invoking the command like this:

actions MarshallPy
{
   $(PYTHON) $(MARSHALLPY) $(>) $(<)
}

So, it should already be utilizing the proper Python interpreter path, as discovered by configure, and not running ./bin/marshall-py.py directly.

We need more information to determine why it is attempting to run the script directly for your case. It might be a good idea to attach the output of the configure run, along with config.log, and the output of your jam invocation, plus any other information which seems relevant.

follow-ups: ↓ 3 ↓ 4   Changed 7 months ago by caedes

Would it still make sense to change the python file header to #/usr/bin/env python ?

Doesnt seem to be fix for this bug, but still probably something nice to do.

if there is nothing against that (And i dont think there should be), ill change it.

in reply to: ↑ 2   Changed 7 months ago by sunshine

Replying to caedes:

Would it still make sense to change the python file header to #/usr/bin/env python ?

There were some old platforms which did not support /usr/bin/env, but I don't think we support any of them any longer, so it probably is safe to make this change.

in reply to: ↑ 2   Changed 7 months ago by sunshine

Replying to caedes:

Doesnt seem to be fix for this bug, but still probably something nice to do.

It would be helpful if we could get additional information and feedback from the original poster of this ticket.

Add/Change #63 (Shebang in marshall-py.py incompatible with mingw)

Author



Action
as new
 
Note: See TracTickets for help on using tickets.