[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: XSLT Transformer plugin



Le Fri, 17 May 2002 01:35:22 +0200, Cyrille Chepelov a écrit:

>Things to check for both plug-ins (where applicable) (may be done, not sure):
>	* GPL headers & copyright notices.
as usual

>	* what XSLT processor does it work with? (this'll need a patch to
>the NEWS file)
done, this is the GNOME xslt processor (it has no dependency on GNOME though)

>	* how does it behave when the XSLT processor isn't here? (we need to
>have dia not depend on the xslt processor, just suggest it). Ideally, we
>gray out the option; at the minimum, we should display an error dialog box.
Use dlopen. But I don't know how to avoid compiling it (using #ifdef's ?)

>	* how does it behave when run from the freshly compiled, but not
>installed tree (as in app/run_dia.sh) ?
this is the only way I tried it yet.

>	* is the string handling UTF-8-clean ? This is a must. At the
>moment, there still must be #hell to handle both a non-UTF-8 gtk (*nix) and
>a modern GTK (Win32).
Not concerned.

>> 	For the commiters: the attached archive contains the stylesheets, 
>> sources and a patch (diff with cvs) for the various Makefile's and 
>> configure changes. This plugin adds a dependency on libxslt. The plugin 
>> assumes that the stylesheets resides in $DIA_PLUGIN_PATH/xslt.
>
>Oh, here's the answer to one question; you add a hard dependency on another
>package. Is it possible/practical to make this a soft dependency (using the
>dl_* functions)? That would make it possible to ship the plug-in in the main
>package. Hmmm. If it's too much effort, don't bother; packagers will have to
>make it a different binary package (from the same source). A little
>inconvenient at first for the package maintainers, but neither unpossible or
>that ugly.
it uses dlopen now.

>
>ObPatchNote: it would be even better if the archive was only a patch (with
>the new files patched against "empty" (there is an option for that in "cvs
>diff")

I tried the option (-N) but I got errors from cvs not founding the CVS/Entries file inside my local directory (which is quite logical :)

Could you tell me the exact way how to do it?


-- 
"Black holes are where God divided by zero." - Steven Wright

MaT|TaM

PGP signature



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index] Mail converted by Mofo Magic and the Flying D

 
All trademarks and copyrights are the property of their respective owners.

Other Directory Sites: SeekWonder | Directory Owners Forum

GuideSMACK