Closed GoogleCodeExporter closed 9 years ago
thanks for the port! That's awesome! I don't know if you have the bandwidth,
but it would be cool if it's possible to get this running on linux/OSX/cygwin
with the same code. I was able to get a single version that was very stable on
linux and osx, but i really don't have much experience with cygwin. I was able
to avoid any one off statements per operating systems, which I think makes it
much more stable. It looks like the main changes were shebang and rerun...if
you can elegantly pull this off, we can incorporate your changes into the main
version.
thanks,
t
Original comment by tobyro...@gmail.com
on 12 Mar 2012 at 10:44
I really enjoyed your talk at PyCON. Your script was so well-written
that was easy to port. I think the line with the env problem might
be a peculiarity of my outdated cygwin installation. Other than that,
there was only a small change to the temporary file path (you had an
extra '/'). Otherwise, the only addition was the 'bs' for backslash
variable. I didn't change "rerun", though, except for the temporary
file path slash issue.
By the way, without cygwin, it is both hopeless and useless to try to
port this to Windows DOS since there is a dearth of commandline
utilities there anyway.
The only other thing I was going to try to add was to enhance the
p.replace() with regular expressions. Finally, and this might be a
cygwin issue, but blank spaces in history[0] don't seem to be working
out for me. This makes it hard to use xargs as the last step in the
pipe chain.
Keep up the great work!
Original comment by recko...@gmail.com
on 13 Mar 2012 at 6:41
thanks...scheduling was pretty hectic for the talk...with I had another day to
prepare! thanks for checking that out. let me investigate the file path. Can
you see if a more modern cygwin will pick up the env shebang? If so, we can
roll out a native version that will run on mac, linux, and windows...
cheers,
t
Original comment by tobyro...@gmail.com
on 14 Mar 2012 at 12:24
It looks like the env problem is deeper than I thought. I upgraded to
the latest version of cygwin and it's still there. I will file a bug
report with the cygwin developers.
Thanks for sharing all your great work!
Original comment by recko...@gmail.com
on 14 Mar 2012 at 6:28
thanks for checking that out. please try this beta. you can use "b" or
"backslash" for split/join operations. also check out the new --quick_output
flag.
Original comment by tobyro...@gmail.com
on 15 Mar 2012 at 11:24
Attachments:
we'll have a special cygwin section for this type of install in the manual
soon...but first verify this works ok.
thanks,
t
Original comment by tobyro...@gmail.com
on 15 Mar 2012 at 11:25
I doesn't work due to the /usr/bin/env problem and the extra '/' in the
tmp file. These are the main changes I made in your orignal script,
besides the 'bs' separator.
Thanks!
Original comment by recko...@gmail.com
on 16 Mar 2012 at 8:11
sorry about that...I'm working on a single beta to fix all issues at
once...please check this one out. It has the temp file fix (i'm surprised this
ran fine on linux and osx actually)...and has a quick output option. shebang
should work for you now, and is the only difference now with the other beta.
Original comment by tobyro...@gmail.com
on 16 Mar 2012 at 9:46
Attachments:
This looks good. I haven't tested all of the functionality, but
everything looks okay so far.
Original comment by recko...@gmail.com
on 17 Mar 2012 at 2:50
new pyp_beta should fix this, but you will have to change the shebang:
http://code.google.com/p/pyp/downloads/detail?name=pyp_beta&can=2&q=#makechanges
Original comment by tobyro...@gmail.com
on 16 May 2012 at 9:38
Original issue reported on code.google.com by
recko...@gmail.com
on 11 Mar 2012 at 7:54Attachments: