git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Convert open("-|") to qx{} calls

From
Randal L. Schwartz <merlyn@stonehenge.com>
Date
Feb 23, 2006, 18:42 UTC
Message-ID
<863bi9hq6u.fsf@blue.stonehenge.com>
In-Reply-To
<7v1wxuhsgw.fsf@assigned-by-dhcp.cox.net>
>>>>> "Junio" == Junio C Hamano <junkio@cox.net> writes:

Junio> The version of Perl Alex has to use claims to be 5.8, but does Junio> not understand open($kid, '-|'), and he is trying to come up Junio> with a workaround.

Ahh, the problem is activestate then. If that's the case, then amend the code with a check for $^O (operating system) that falls back to a qx if on activestate, and hope that filenames aren't a problem. Unfortunately, I don't know enough about that to fix it.

But whatever you do, *don't* replace safe_qx with qx() for all other systems, or you'll be opening up a can of worms for those of us on sensible systems.

-- 
Randal L. Schwartz - Stonehenge Consulting Services, Inc. - +1 503 777 0095
<merlyn@stonehenge.com> <URL:http://www.stonehenge.com/merlyn/>
Perl/Unix/security consulting, Technical writing, Comedy, etc. etc.
See PerlTraining.Stonehenge.com for onsite and open-enrollment Perl training!
Previous: Junio C HamanoNext: Johannes Schindelin
Message 5 of 13 in “Convert open("-|") to qx{} calls”
  1. Convert open("-|") to qx{} callsJohannes Schindelin, Feb 23, 2006
  2. Alex RiesenFeb 23, 2006
  3. Randal L. SchwartzFeb 23, 2006
  4. Junio C HamanoFeb 23, 2006
  5. Randal L. SchwartzFeb 23, 2006
  6. Johannes SchindelinFeb 23, 2006
  7. Randal L. SchwartzFeb 23, 2006
  8. Alex RiesenFeb 23, 2006
  9. Randal L. SchwartzFeb 23, 2006
  10. Rogan DawesFeb 24, 2006
  11. Alex RiesenFeb 24, 2006
  12. Rogan DawesFeb 24, 2006
  13. Alex RiesenFeb 24, 2006

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.