Re: patch for tmux on cygwin

2013-07-15 Thread Thomas Adam
On 15 July 2013 22:07, Saad Malik wrote: > Mark, I don't know--maybe Raynor or Nicholas can chime in. > > Try running tmux with 'strace tmux'. strace -f -o /somewhere/tmux.out tmux and send through the log file, please. -- Thomas Adam ---

Re: patch for tmux on cygwin

2013-07-15 Thread Saad Malik
Mark, I don't know--maybe Raynor or Nicholas can chime in. Try running tmux with 'strace tmux'. On Mon, Jul 15, 2013 at 12:15 PM, Mark Volkmann wrote: > Thanks for the detailed instructions! > I got everything to build without errors using these steps. > I have the tmux command now. > > However

Re: patch for tmux on cygwin

2013-07-15 Thread Mark Volkmann
Thanks for the detailed instructions! I got everything to build without errors using these steps. I have the tmux command now. However, when I enter "tmux new-session -s foo" it runs for a couple of seconds and nothing happens. It doesn't start a tmux session. On Mon, Jul 15, 2013 at 12:52 PM, S

Re: patch for tmux on cygwin

2013-07-15 Thread Saad Malik
Mark, Okay, at least you have the sources now. *libevent* $ cd libevent-2.0.21-stable $ make clean $ ./configure --prefix=/usr $ make && make install *tmux* $ cd tmux-tmux-code $ make clean $ CFLAGS="-I/usr/include/ncurses" ./configure --prefix=/usr $ make && make install Let me know how this g

Re: patch for tmux on cygwin

2013-07-15 Thread Mark Volkmann
I just tried the steps you suggested. The "make install" for libevent fails for me. It says the following: Makefile:693: recipe for target 'libevent.la' failed Makefile:857: recipe for target 'install-recursive' failed Makefile:1182: recipe for target 'install' failed On Fri, Jul 12, 2013 at 8:4

Re: patch for tmux on cygwin

2013-07-12 Thread Saad Malik
Mark, I don't know why. That works perfectly for me. Make sure you don't have a '.tmux.conf' that might be causing issues? Last try, I've tarred up the source folders for libevent and tmux: https://docs.google.com/file/d/0B7oqPEemRewhaVlmNTlKSHdQTTQ 1. Untar to any directory (tar -xzf tmux-libeven

Re: patch for tmux on cygwin

2013-07-12 Thread Mark Volkmann
Thanks! No errors now, but when I enter: tmux new-session -s mytmux it runs for a few seconds and then ends without actually starting tmux. I do have ncurses installed. On Fri, Jul 12, 2013 at 7:51 PM, Saad Malik wrote: > Mark, > I've added the libevent libraries to the binary. Please make su

Re: patch for tmux on cygwin

2013-07-12 Thread Saad Malik
Mark, I've added the libevent libraries to the binary. Please make sure that you have the ncurses dependency also installed (via cygwin setup.exe). Updated package: https://docs.google.com/file/d/0B7oqPEemRewhX2hXRzNrYS1EUms saamalik@SAAMALIK-WS02 ~ $ md5sum.exe tmux-cygwin-13-07-12.tar.gz 26b538

Re: patch for tmux on cygwin

2013-07-12 Thread Saad Malik
My apologies. I forgot to include the libevent dependency for tmux. Give me a couple minutes and I'll upload a new package. -Saad On Fri, Jul 12, 2013 at 5:26 PM, Mark Volkmann wrote: > Thanks for uploading that! When I run it I get this: > > /usr/bin/tmux.exe: error while loading shared librar

Re: patch for tmux on cygwin

2013-07-12 Thread Mark Volkmann
Thanks for uploading that! When I run it I get this: /usr/bin/tmux.exe: error while loading shared libraries: cygevent-2-0-5.dll: can not open shared object file: No such file or directory Are there other things I need to install to get it to work? On Fri, Jul 12, 2013 at 5:02 PM, Saad Malik w

Re: patch for tmux on cygwin

2013-07-12 Thread Saad Malik
Sure, no problem. Uploaded tmux cygwin binary here: https://docs.google.com/file/d/0B7oqPEemRewhUDlkbWJhbDlEY0E $ md5sum.exe tmux-cygwin-13-07-12.tar.gz 81855a64c2bf53fdbadda5e73b2105dc *tmux-cygwin-13-07-12.tar.gz $ tar -tzf tmux-cygwin-13-07-12.tar.gz usr/bin/tmux.exe usr/share/man/man1/tmux.1

Re: patch for tmux on cygwin

2013-07-12 Thread Patrick Shanahan
* Saad Malik [07-12-13 17:43]: > Mark, I've been using tmux on Cygwin for a whole day--all the features > work, it's stable, and fast! You have to compile using the cygwin gcc > toolkit--if you want I can upload a binary for you. providing a binary would be very nice of you. Althought I run tmux

Re: patch for tmux on cygwin

2013-07-12 Thread Mark Volkmann
An uploaded binary would be great! On Fri, Jul 12, 2013 at 4:41 PM, Saad Malik wrote: > Mark, I've been using tmux on Cygwin for a whole day--all the features > work, it's stable, and fast! You have to compile using the cygwin gcc > toolkit--if you want I can upload a binary for you. > > > On F

Re: patch for tmux on cygwin

2013-07-12 Thread Saad Malik
Mark, I've been using tmux on Cygwin for a whole day--all the features work, it's stable, and fast! You have to compile using the cygwin gcc toolkit--if you want I can upload a binary for you. On Fri, Jul 12, 2013 at 2:32 PM, Mark Volkmann wrote: > Is tmux on cygwin considered ready to use now?

Re: patch for tmux on cygwin

2013-07-12 Thread Mark Volkmann
Is tmux on cygwin considered ready to use now? If so, are there notes on the installation steps somewhere? On Fri, Jul 12, 2013 at 4:26 PM, Nicholas Marriott < nicholas.marri...@gmail.com> wrote: > I've applied this now, thanks for your work! > > > On Wed, Jul 10, 2013 at 03:49:19PM -0500, J Ray

Re: patch for tmux on cygwin

2013-07-12 Thread Nicholas Marriott
I've applied this now, thanks for your work! On Wed, Jul 10, 2013 at 03:49:19PM -0500, J Raynor wrote: > > Are you sure ECHOPRT 0x20 is right on Cygwin? Maybe it should be 0 to > > leave it out if it isn't supported? > > I'm not the least bit sure, so I'll defer to you. > > I tested your patch

Re: patch for tmux on cygwin

2013-07-10 Thread Mark Volkmann
Please do submit this as a cygwin package! I am SO looking forward to using this! On Wed, Jul 10, 2013 at 4:31 PM, Saad Malik wrote: > J Raynor/Nicholas! Awesome job, and thank you! The latest patch by > Nicholas applied cleanly, and I'm running tmux in cygwin! Woot =) > > 1. As J Raynor mentio

Re: patch for tmux on cygwin

2013-07-10 Thread Saad Malik
J Raynor/Nicholas! Awesome job, and thank you! The latest patch by Nicholas applied cleanly, and I'm running tmux in cygwin! Woot =) 1. As J Raynor mentioned, the CFLAGS is required for ncurses header to be found: CFLAGS="-I/usr/include/ncurses" ./configure --prefix=/usr. Can this be done automati

Re: patch for tmux on cygwin

2013-07-10 Thread J Raynor
> Are you sure ECHOPRT 0x20 is right on Cygwin? Maybe it should be 0 to > leave it out if it isn't supported? I'm not the least bit sure, so I'll defer to you. I tested your patch and it worked fine. -- See everything fr

Re: patch for tmux on cygwin

2013-07-05 Thread Nicholas Marriott
Are you sure ECHOPRT 0x20 is right on Cygwin? Maybe it should be 0 to leave it out if it isn't supported? Also some other minor tweaks. diff --git a/client.c b/client.c index 691ace3..b9e3b30 100644 --- a/client.c +++ b/client.c @@ -328,8 +328,13 @@ client_send_identify(int flags) str

Re: patch for tmux on cygwin

2013-06-02 Thread J Raynor
I've attached a new patch that puts the msg_identify_data change in an ifdef. Also, in compat.h, I moved the "undef b64_ntop" for cygwin into the "ifndef HAVE_B64_NTOP" check so that if cygwin ever gets a working b64_ntop tmux will continue to compile. If I do "tmux new tmux-cygwin.patch Descri

Re: patch for tmux on cygwin

2013-05-31 Thread Nicholas Marriott
Thanks. We can't change msg_identify_data like this without bumping the protocol version, so that bit needs to go under #ifdef too. What happens if you do "tmux new I've attached an updated patch. > > You had wanted the change to resolv.h to go into compat.h, but I > didn't see how to do that s

Re: patch for tmux on cygwin

2013-05-20 Thread J Raynor
I've attached an updated patch. You had wanted the change to resolv.h to go into compat.h, but I didn't see how to do that since I needed to comment it out. But I figured out that I just needed to undef something. I put that in compat.h, so now tty.c isn't changed at all. tmux-cygwin.patch Des

Re: patch for tmux on cygwin

2013-05-18 Thread J Raynor
I've attached a new patch. TTY_NAME_MAX is already defined in compat.h, so I didn't need to add it. Gcc didn't define _WIN32, so I used __CYGWIN__ instead. Regarding this: > You should probably fail early if isatty is false (before filling in ttyname > into the identify struct). I believe isa

Re: patch for tmux on cygwin

2013-05-18 Thread Nicholas Marriott
On Sat, May 18, 2013 at 03:35:44AM -0500, J Raynor wrote: > The patch I had sent in was meant for testing, and to see if there was > any interest, so I wasn't putting any effort into getting it into > shape for acceptance. Now that that might be possible, I'll get it > cleaned up, taking into acco

Re: patch for tmux on cygwin

2013-05-18 Thread J Raynor
The patch I had sent in was meant for testing, and to see if there was any interest, so I wasn't putting any effort into getting it into shape for acceptance. Now that that might be possible, I'll get it cleaned up, taking into account your notes. > I guess this obviously won't let other things

Re: patch for tmux on cygwin

2013-05-18 Thread Nicholas Marriott
Thanks - this looks much better, we can probably do something with this. I guess this obviously won't let other things that need file descriptor passing work like "loadb -" or "saveb -". A few things: Why do you need to dup stdin in the client? Can you replace cyg_get_identify_fd by just puttin

Re: patch for tmux on cygwin

2013-05-17 Thread J Raynor
I've found a better way to get tmux working on cygwin. There's now no need to modify tty.c to have separate read and write file descriptors. I've attached the new patch. On Wed, May 15, 2013 at 12:59 AM, J Raynor wrote: > Below is a patch for getting tmux running on cygwin. I only have acce

Re: patch for tmux on cygwin

2013-05-14 Thread J Raynor
I've added the patch as an attachment since it seems it was garbled the first time. On Wed, May 15, 2013 at 1:32 AM, Nicholas Marriott < nicholas.marri...@gmail.com> wrote: > Hi > > Your mailer has mangled the patch, can you attach it? > > > On Wed, May 15, 2013 at 12:59:56AM -0500, J Raynor wro

Re: patch for tmux on cygwin

2013-05-14 Thread Nicholas Marriott
Hi Your mailer has mangled the patch, can you attach it? On Wed, May 15, 2013 at 12:59:56AM -0500, J Raynor wrote: >Below is a patch for getting tmux running on cygwin.* I only have access >to a 32 bit XP system, so I don't know if it works on other versions of >Windows.* Cygwin does

patch for tmux on cygwin

2013-05-14 Thread J Raynor
Below is a patch for getting tmux running on cygwin. I only have access to a 32 bit XP system, so I don't know if it works on other versions of Windows. Cygwin doesn't provide a libevent package, so you'll first have to install that. I used libevent-2.0.21-stable. Also, for tmux I had to specif