Re: [zathura] Synctex support

2012-03-06 Thread Roland Schatz
Hi > As zathura has undergone major changes we can't just merge it. Are you > interested in porting synctex support to 0.1.0? If you need any help I'm > available to guide you through the new code. I'm still interested. I'll have a look at the new code to see how to port it. The backwards synchro

Re: [zathura] Synctex support

2012-04-13 Thread Roland Schatz
> Although I don't think I would be able to implement that, I would > warmly recommend against adopting unneeded complexity layers like > dbus, which is deployed, e.g., for evince's syntex support. > I'm sure that working with command line arguments, like the good old > xkdvi does, would be just be

[zathura] [PATCH] synctex backwards syncronization

2012-07-10 Thread Roland Schatz
I have now finished porting at least backwards synchronization to the new version of zathura. Patch attached. Usage is as I described in the earlier mail: -s: enables synctex support -x : sets the synctex editor (optional, see man synctex) When synctex is enabled, ctrl-leftclick on the page wil

Re: [zathura] [PATCH] synctex backwards syncronization

2012-07-13 Thread Roland Schatz
Hi, > * Could you please use g_strdup_printf instead of a fix sized buffer? > * For consistency with all the other parts in girara and zathura that spawn > process please use something like g_spawn_sync to execute synctex. No problem. I should really make myself familiar with glib, there are some q

Re: [zathura] [PATCH] synctex backwards syncronization

2012-07-13 Thread Roland Schatz
> One more think that just came to my mind: could you document -s and -x in > zathura.1.rst? A section on how to use synctex would be nice there as well. Ok. I'm not sure if I'll get to it this weekend, but I should have some time next week. That reminds me: I forgot to mention that the two new st

Re: [zathura] [PATCH] synctex backwards syncronization

2012-08-13 Thread Roland Schatz
I just noticed you removed the -s option. That raises a few design questions: First of all, when removing -s, the "zathura->synctex.enabled" configuration variable becomes obsolete. The question now becomes, how do you enable synctex? I see two options (not counting the original solution with the

[zathura] [PATCH] synctex forward synchronization

2012-08-14 Thread Roland Schatz
I have attached the patch for forward synchronization. This patch is a lot more complex than the previous one, and I haven't really tested it rigorously (although I installed it on my production system today). It introduces a new command ":synctex ::" (same syntax than the "synctex view -i" param