On Tue, Aug 29, 2023 at 05:50:12PM +0200, Pavel Sanda wrote: > On Tue, Aug 29, 2023 at 05:27:16PM +0200, Jürgen Spitzmüller wrote: > > Am Dienstag, dem 29.08.2023 um 17:23 +0200 schrieb Pavel Sanda: > > > I am afraid this will break on the problems I tried to handle inside > > > the script and will complicate the situation if you want to slip your > > > own scripts in the pipeline. > > > > Like which problems exactly? > > - Not waiting for a viewer to finish & spawn multiples instances of viewer > when more result on one query is found > - eat STDERR output to avoid mysterious crashes of spawned processes > - allow specific arguments for specific viewers > > > I find it much more straighforward, and maybe also more safe. Should > > the external script really do more than query and return found file > > paths? > > Well obviously that's the question - who is responsible for launching > the viewer ;) > > Anyway I am not totally oppposed to your approach, but I'll need more time to > check what got broken...
Quick check shows that the new approach breaks in multiple ways on my system. I'm sure there are ways to gradually fix this but I have different proposal. Would you mind that instead of just checking for empty results we allow the script also to return e.g. "?internally_handled?" and show no response in such case? This way you can stay with elegant and safe world while I can stay in a dangerous one without turning this thread into saga of corner cases? :) Pavel -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailman/listinfo/lyx-devel
