[rfc][icedtea-web] IcedTea-Web for Windows
Jiri Vanek
jvanek at redhat.com
Thu Sep 11 10:03:52 UTC 2014
On 09/11/2014 03:38 AM, Jacob Wisor wrote:
> Hello there!
>
> I was thinking about bringing IcedTea-Web to Windows for quite some time now. So I have looked into
> the current plug-in code for Un*x systems. Unfortunately, it has dependencies on pthreads and GTK+.
> Although these could be compiled in statically, this would actually not solve all the problems to it.
> Mainly, three things would not work.
> First, error logging. Because Windows has slightly different pipe and IPC semantics to POSIX the
> current code makes to many assumptions on their behavior while only applicable to POSIX. Although I
> am pretty sure the GTK+ library tries to resemble POSIX semantics as close as possible, or abstract
> them away with its concept of channels, I am also sure that we would run into some bogus and hard to
> track down bugs later on. Error logging from Java applications has to be designed on Windows quite
> differently.
>
> Second, sub-process shutdown. Windows has some quite different process semantics than POSIX.
> Terminating the JVM in-process server in Windows properly requires really Windows specific code.
> Ports of pthreads and GTK+ won't do much here.
>
> Third, plug-in registration, detection, and install/uninstall. These tasks involve the Windows
> registry, which of course is highly Windows specific. GTK+ has nothing to offer here. And lastly (or
> forth), getting the plug-in on the Internet Explorer is absolutely impossible without registering
> and providing a scriptable COM component. While Mozilla compatible browsers can load plug-ins from
> designated folders, the Internet Explorer relies entirely on the OLE server and COM infrastructure,
> so registration is required. So, why bother about IE anyway? Well, it's like the No. 1 browser on
> the Windows platform. Without support for IE, most people will have even less incentive to use
> open-source IcedTea-Web.
>
> Long story short, I came to the conclusion that it is probably best to start implementing Windows
> specific code now and to blend it with the existing Un*x code later.
Do you think it is even possible?
>
> At this stage, the plug-in does not do much, except for registering the plug-in with Mozilla
> compatible browsers and COM. However, you won't see it in the list of Add-ons in IE. Simply because
> IE requires an implemented IOleObject interface, which I did not come to yet. ;-) But, the plug-in
> does show up in Mozilla compatible browsers! I have been trying to test the Chrome browser too but
> to no avail. So far, I have been building and testing the 64 bit version only.
>
> How to build
>
> True to IcedTea's motto, you will not need any proprietary or closed source tools to build. However,
> you will need MinGW, mingw-make, XulRunner SDK (a Linux package works too), and OpenJDK with Windows
> headers. Actually, I have been coding in F20 entirely, and the only thing I had to pull manually
> were the JVM Windows headers, hence you should be good on almost every Linux distro.
> Before building, look into the Makefile.mingw file to setup your paths or to find out about the make
> variables to set on the command line. Then invoke
>
> mingw-make -C plugin/icedteanp/windows -f Makefile.mingw all
>
> or for the 64 bit version
>
> mingw-make -C plugin/icedteanp/windows -f Makefile.mingw \
> PROCESSOR_ARCHITECTURE=AMD64 all
>
> I have designed the make file to build on Un*x and Windows systems, although I have not tested
> building on Windows yet.
>
> Have fun and thank you for reviewing!
>
Oh this is magnificent chnageset!
I have absolutely nothing about windows specific part of this patch. I'm also compltly out of skill
on this field. So the sources of this part are (from my side) always ok to go freely.
Some comments to
>
> IcedTea-Web DLL properties.png
>
>
>
> IcedTea-Web in Mozilla Nigtly x64 for Windows.png
>
>
>
> Windows plug-in.patch
>
>
> diff --git a/.hgignore b/.hgignore
> --- a/.hgignore
> +++ b/.hgignore
> @@ -1,13 +1,23 @@
> -Makefile.in
> -aclocal.m4
> -autom4te.cache
> +Makefile\.in
> +aclocal\.m4
> +autom4te\.cache
> build
> configure
> install-sh
> missing
> -config.guess
> -config.sub
> -netx/net/sourceforge/jnlp/resources/AUTHORS.html
> -netx/net/sourceforge/jnlp/resources/COPYING.html
> -netx/net/sourceforge/jnlp/resources/ChangeLog.html
> -netx/net/sourceforge/jnlp/resources/NEWS.html
> +config\.guess
> +config\.sub
> +.+\.patch
> +.+\.[Rr][Ee][Ss]
> +.+\.(o|[Oo][Bb][Jj])
> +.+\.[Ee][Xx][Ee]
> +.+\.(so|[Dd][Ll]{2})
> +.+\.(a|[Ll][Ii][Bb])
> +.+\.[Mm][Ss][Ii]
> +.+\.rpm
> +.+\.[Xx][Pp][Ii]
> +(\.settings|\.project|\.cproject)
> +netx/net/sourceforge/jnlp/resources/AUTHORS\.html
> +netx/net/sourceforge/jnlp/resources/COPYING\.html
> +netx/net/sourceforge/jnlp/resources/ChangeLog\.html
> +netx/net/sourceforge/jnlp/resources/NEWS\.html
Except the .project this part is ok to go as separate changeset.
.project, is legacy habit to include classapth code style modules for eclipse in main trunk.
Personally I'm for removing those, and uplaod them for download to
http://icedtea.classpath.org/wiki/IcedTea-Web#Code_style
Once uploaded and deleted, .project can be excluded.
> diff --git a/javaws.ico b/javaws.ico
> new file mode 100644
> index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..d8f848b89ffe5b7986e83e1d4d0a250c40ac6afb
Why this? The ico is same as javaws.png. So I would inlicne to runtime generation from jaavws.png to
javaws.ico.
Also - why not to use png?
Ifabove seems to be unacceptable, then I would move this icon to windows folder.
> GIT binary patch
> literal 77494
> zc%1EB2S5`^7vA&)2uVmn at 4a^r1gT<|rhrtj04f$#P!SPSR1`s^BX-4J(6cLwsMxWc
> zo?Yx<LF@=f{@H8 at A%sv2sQ2HO-OSGJ?CjgO)91Z65QKv85S0o6R~8zFfuL{*f at Eb8
...snip...
> zJ3&{NHu;q)P3aebq6(`8!(@>R#C?BGW-Mv)BI&o8ekA=jr${9KG(UjCVzqbxk{_GD
> zpd-iM^uPo at KgZid0E*9#;qw#P@>lTrYg&-k+F#o-zY3qf>2;bIkkC<oW6RfV*N-p=
> z)s7$2>@R8Y`fd0r&Hjo``fFOSqqV=b1$#IQNNC63>_r&Rj-TiC*(pq at Kd}XScnbG6
> z{1q);w?lpvUVlw%zay`|me1eNf*tG^WVJxc*Xe*C)6$>F=Px0&^jGlvDJ}hQN!Y^g
> Q*wSCi?@ti!Z)o!W4+)&_H~;_u
>
> diff --git a/plugin/icedteanp/windows/GUIDs.c b/plugin/icedteanp/windows/GUIDs.c
> new file mode 100644
> --- /dev/null
> +++ b/plugin/icedteanp/windows/GUIDs.c
...
> diff --git a/plugin/icedteanp/windows/IcedTea-Web.h b/plugin/icedteanp/windows/IcedTea-Web.h
> new file mode 100644
> --- /dev/null
> +++ b/plugin/icedteanp/windows/IcedTea-Web.h
....
> diff --git a/plugin/icedteanp/windows/IcedTea-Web.rc b/plugin/icedteanp/windows/IcedTea-Web.rc
> new file mode 100644
> --- /dev/null
> +++ b/plugin/icedteanp/windows/IcedTea-Web.rc
...Well.. What to say.. Really magnificent.
...
thoseˇ
> +#endif /* HAVE_JAVA8 */
> +
> +#ifndef ICEDTEA_WEB_MIME_TYPES
> +#define ICEDTEA_WEB_MIME_TYPES \
> + "application/x-java-applet|" \
> + "application/x-java-applet|" \
> + "application/x-java-applet;version=1.1|" \
> + "application/x-java-applet;version=1.1|" \
> + "application/x-java-applet;version=1.1.1|" \
> + "application/x-java-applet;version=1.1.1|" \
> + "application/x-java-applet;version=1.1.2|" \
>...
> + "application/x-java-bean;version=1.6|" \
> + "application/x-java-bean;version=1.6|" \
> + "application/x-java-bean;version=1.7|" \
> + "application/x-java-bean;version=1.7|" \
> + PLUGIN_BEAN_MIME_DESC \
> + PLUGIN_BEAN_MIME_DESC \
...those...
> +// "class|jar|"
> +// "class|jar|"
> +// "class|jar|"
> +// "class|jar|"
> +// "class|jar|"
> +// "class|jar|"
> +// "class|jar|"
> +// "jnlp|"
> +// "class|jar"
> +// VALUE "FileOpenName", "Java-Klassendatei (*.class)|"
> +// "Java-Archiv (*.jar)|"
> +// "Java-Klassendatei (*.class)|"
> +// "Java-Archiv (*.jar)|"
> +// "Java-Klassendatei (*.class)|"
> +// "Java-Archiv (*.jar)|"
> +// "Java-Klassendatei (*.class)|"
> +// "Java-Archiv (*.jar)|"
...those...
> + "class|jar|"
> + "class|jar|"
> + "class|jar|"
> + "class|jar|"
> + "class|jar|"
> + "jnlp|"
> + "class|jar"
> + VALUE "FileOpenName", "Java Class File (*.class)|"
> + "Java Archive (*.jar)|"
> + "Java Class File (*.class)|"
> + "Java Archive (*.jar)|"
> + "Java Class File (*.class)|"
> + "Java Archive (*.jar)|"
> + "Java Class File (*.class)|"
> + "Java Archive (*.jar)|"
> + "Java Class File (*.class)|"
> + "Java Archive (*.jar)|"
> + "Java Class File (*.class)|"
...those...
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)|"
> +// "Aplikacja JNLP (*.jnlp)|"
> +// "Plik Klassy Java (*.class)|"
> +// "Archiwum Java (*.jar)"
> +// VALUE "Language", "pl"
> +// }
> + }
and those ^ seems to me like most easy to be belnded. Together with ITW
But defintiely not as this changeset.
> +
> + BLOCK "VarFileInfo" {
> + VALUE "Translation", LANG_CZECH, 1250, /* Czech */
> + LANG_GERMAN, 1252, /* German */
> + LANG_ENGLISH, 1252, /* English */
> + LANG_POLISH, 1250 /* Polish */
> + }
> +}
> +
> +ID_ICON ICON "../../../javaws.ico"
> +
> +STRINGTABLE LANGUAGE LANG_CZECH, SUBLANG_DEFAULT {
> + IDS_FRIENDLYTYPENAME "JNLP aplikace"
> +}
> +
> +STRINGTABLE LANGUAGE LANG_ENGLISH, SUBLANG_DEFAULT {
> + IDS_FRIENDLYTYPENAME "JNLP Application"
> + IDS_OPEN "&Open"
> +}
> +
> +STRINGTABLE LANGUAGE LANG_GERMAN, SUBLANG_DEFAULT {
> + IDS_FRIENDLYTYPENAME "JNLP-Anwendung"
> + IDS_OPEN "Ö&ffnen"
> +}
> +
> +STRINGTABLE LANGUAGE LANG_POLISH, SUBLANG_DEFAULT {
> + IDS_FRIENDLYTYPENAME "Aplikacja JNLP"
> + IDS_OPEN "O&twórz"
> +}
> +
> +#else /* OS2 */
> +#include <npapi.h>
> +
> +NP_INFO_ProductVersion RC_DATA {
> + PACKAGE_MAJOR_VERSION,
> + PACKAGE_MINOR_VERSION,
> + PACKAGE_PATCH_VERSION,
> + PACKAGE_BUILD_VERSION
> +}
> +NP_INFO_MIMEType RC_DATA {
> + ICEDTEA_WEB_MIME_TYPES "\0"
> +}
> +NP_INFO_FileOpenName RC_DATA {
> + "Java Class File (*.class)|"
> + "Java Archive (*.jar)|"
> + "JNLP Application (*.jnlp)|\0"
> +}
> +NP_INFO_FileExtents RC_DATA {
> + "class|jar|jnlp\0"
> +}
> +NP_INFO_FileDescription RC_DATA {
> + PACKAGE_DESCRIPTION "\0"
> +}
> +NP_INFO_ProductName RC_DATA {
> + PACKAGE_NAME "\0"
> +}
> +NP_INFO_CompanyName RC_DATA {
> + PACKAGE_VENDOR "\0"
> +}
> +NP_INFO_FileVersion RC_DATA {
> + PACKAGE_MAJOR_VERSION,
> + PACKAGE_MINOR_VERSION,
> + PACKAGE_PATCH_VERSION,
> + PACKAGE_BUILD_VERSION
> +}
> +NP_INFO_InternalName RC_DATA {
> + "NP" PACKAGE_NAME "\0"
> +}
> +NP_INFO_LegalCopyright RC_DATA {
> + "GPLv2, see the file COPYING for details\0"
> +}
> +NP_INFO_OriginalFilename RC_DATA {
> + "NP" PACKAGE_NAME ".DLL\0"
> +}
> +#endif /* OS2 */
> +#endif /* RC_INVOKED */
> +#endif /*_ICEDTEA_WEB_RC_ */
> diff --git a/plugin/icedteanp/windows/IcedTeaIEPlugin.c b/plugin/icedteanp/windows/IcedTeaIEPlugin.c
> new file mode 100644
> --- /dev/null
> +++ b/plugin/icedteanp/windows/IcedTeaIEPlugin.c
> @@ -0,0 +1,2191 @@
> +#define INC_OLE2
> +#include <windows.h>
> +#include <olectl.h>
... http://en.wiktionary.org/wiki/to_je_pro_m%C4%9B_%C5%A1pan%C4%9Blsk%C3%A1_vesnice :))
> +}
> diff --git a/plugin/icedteanp/windows/IcedTeaIEPlugin.h b/plugin/icedteanp/windows/IcedTeaIEPlugin.h
> new file mode 100644
> --- /dev/null
> +++ b/plugin/icedteanp/windows/IcedTeaIEPlugin.h
> @@ -0,0 +1,3 @@
....still the same.
> \ No newline at end of file
>
Well one general hint - Maybe its the time to restructure plugin directory?
I would go for something like
plugin/windows/src (your "c a nd rest" files)
plugin/windows/build (mingw make or whatever you may need for build (the ico if you will insists?-) ) ))
plugin/linux/src (current icedteanp)
plugin/linux/build? And so extract plugin parts from main makefile?
plugin/share/java (current current java)
plugin/share/docs
plugin/share/tests
This si just idea, not necessary for this changeset or at all. And can be complicated to do...
J.
More information about the distro-pkg-dev
mailing list