Re: Test file for the embedding feature.

2008-01-09 Thread Bo Peng
> Why not using LyX.Embed.X for the drive letter under windows? The logic in EmbeddedFIle::set() should be carefully examined. It works like this now, 1. When a file is created, for example, document path: /home/bpeng/doc/file.lyx embedded file: /home/bpeng/figures/figure.png Its relative

Re: Test file for the embedding feature.

2008-01-09 Thread Bo Peng
> I understand the path problem, but not why LyX crashes. When an embedded file > could not be found (no > matter if the path or what ever is the reason), LyX should display the error > message as it already > does, but then not crash. In the implementation, I intentionally assumes the availabil

Re: Test file for the embedding feature.

2008-01-09 Thread Uwe Stöhr
> The problem is clear: there is no unique representation of absolute > path under different OS. I understand the path problem, but not why LyX crashes. When an embedded file could not be found (no matter if the path or what ever is the reason), LyX should display the error message as it already

Re: Test file for the embedding feature.

2008-01-08 Thread Abdelrazak Younes
Bo Peng wrote: In the end, Joost's suggestion might be the only solution. Joost's solution has its own features/problems. For example, when I insert the same file twice, the current solution consider them as the same file (both external and inzip). However, using UUID as inzip filename, they wi

Re: Test file for the embedding feature.

2008-01-08 Thread Abdelrazak Younes
Bo Peng wrote: The problem is clear: there is no unique representation of absolute path under different OS. The absolute file in the test lyx file is /home/bpeng/blah and its inzip name is LyX.Embed.Abs/home/bpeng/blah. Under windows, the file is considered as c:/home/bpeng/blah and its inzip na

Re: Test file for the embedding feature.

2008-01-08 Thread Bo Peng
> In the end, Joost's suggestion might be the only solution. Joost's solution has its own features/problems. For example, when I insert the same file twice, the current solution consider them as the same file (both external and inzip). However, using UUID as inzip filename, they will be different.

Re: Test file for the embedding feature.

2008-01-08 Thread Bo Peng
The problem is clear: there is no unique representation of absolute path under different OS. The absolute file in the test lyx file is /home/bpeng/blah and its inzip name is LyX.Embed.Abs/home/bpeng/blah. Under windows, the file is considered as c:/home/bpeng/blah and its inzip name is LyX.embed.A

Re: Test file for the embedding feature.

2008-01-08 Thread Bo Peng
On Jan 8, 2008 5:38 PM, Uwe Stöhr <[EMAIL PROTECTED]> wrote: > Bo Peng schrieb: > > > Did you see this right after lyx opens the file? > > Yes. OK, I am building a windows version. Bo

Re: Test file for the embedding feature.

2008-01-08 Thread Uwe Stöhr
Bo Peng schrieb: Did you see this right after lyx opens the file? Yes. Uwe

Re: Test file for the embedding feature.

2008-01-08 Thread Bo Peng
> I get: > > Failed to embed file C:/usr/local/share/icons/hicolor/48x48/apps/poedit.png. > Please check whether this file exists and is readable. > > Because I don't have the poedit.png. > But then LyX crashes! I know absolute path would lead to problems... Did you see this right after lyx opens

Re: Test file for the embedding feature.

2008-01-08 Thread Uwe Stöhr
> I have uploaded a test file to > http://www.lyx.org/~bpeng/test.lyx, which has several embedded insets. > Please play with it and see if you like how embedding is done > now. I get: Failed to embed file C:/usr/local/share/icons/hicolor/48x48/apps/poedit.png. Please check whether this file exis

Re: Test file for the embedding feature.

2008-01-08 Thread Bo Peng
> Maybe this has been discussed before but why don't we standardize on a > .zlyx or .lyz extension? If you are talking about blocking, extension does not matter. In general, the problem of .lyz is that there is no easy logic for 'save' and 'save as' if two file extensions are used. Bo

Re: Test file for the embedding feature.

2008-01-08 Thread Abdelrazak Younes
Bo Peng wrote: Dear all, With my last several commits, I have added embedding support for InsetGraphics, InsetInclude, InsetExternal and InsetBibtex (any more??). I have uploaded a test file to http://www.lyx.org/~bpeng/test.lyx, which has several embedded insets. Please play with it and see if

Test file for the embedding feature.

2008-01-08 Thread Bo Peng
Dear all, With my last several commits, I have added embedding support for InsetGraphics, InsetInclude, InsetExternal and InsetBibtex (any more??). I have uploaded a test file to http://www.lyx.org/~bpeng/test.lyx, which has several embedded insets. Please play with it and see if you like how emb