25068 - Need some sample plugin-in code under Linux

Status: VERIFIED FIXED

(Core Graveyard :: Plug-ins, defect, P3)

Description

[Jerry.Kirk]

We would like to start integrating existing Photon plugins into Mozilla
and can not find any sample Linux/Unix plug-in code anywhere.

Comment 1

[Kathleen :Brade]

QA will need a plugin to do their testing with also.  This could be the same 
plugin.

CC leger since this is a blocker for her team (tester plugin)

Comment 2

[Jerry.Kirk]

I was pointed to modules/plugin/src but that will not compile because it is
missing header files:
In file included from nsplugin.cpp:31:
npglue.h:41: nsICapsManager.h: No such file or directory
In file included from nsplugin.cpp:31:
npglue.h:53: layout.h: No such file or directory
npglue.h:56: layers.h: No such file or directory
nsplugin.cpp:62: nsICapsManager.h: No such file or directory
nsplugin.cpp:63: nsCCapsManager.h: No such file or directory
In file included from npglue.h:45,
                 from nsplugin.cpp:31:
nppriv.h:119: syntax error before `*'
In file included from npglue.h:51,
                 from nsplugin.cpp:31:

Comment 3

[av (gone)]

Code in plugin/src has nothing to do with new plugin API and is not used in the 
build process. If you are looking for the sample code from the old world you can 
download Plugin SDK. Some sample code for the new API is sitting in 
modules/plugin/test. Although I am not sure how cross-platform it is.

Comment 4

[Eric Pollmann]

The Plugins SDK (including the Unix SDK) for the old style plugins is here:

http://home.netscape.com/comprod/development_partners/plugin_api

And for the record, I think the code in modules/plugin/test is pretty XP, I got
it to compile with just adding a #include nsString2.h on Linux.  :)

Comment 5

[Jerry.Kirk]

I appreciate all the action on this bug. I guess I consider it closed since
some Linux sample code was pointed out to me.. I will try it under Linux and
possibly convert it over from GTK to Photon. If I have anymore trouble I will
file a new bug.

Comment 6

[Jerry.Kirk]

I looked at this a little more... and this "Simple" example plugin seems
to require Java which I do not have on this platform yet... Is JAVA really
a requirement for a simple example plugin?

Comment 7

[ekrock's old account (dead)]

A non-Netscape participant may be able to provide sample code here. Do not start
development of this sample code without checking with me to see if that worked
out; we don't want to duplicate effort.

Comment 8

[rusty.lynch]

I have a patch for the current XPCom sample plugin, npsimple, to make it work
as much as on a Windows box.  I will attach the patch to this bug report.

Comment 9

[rusty.lynch]

Attachment: apply to mozilla/modules/plugin/test/npsimple.cpp

Comment 10

[rusty.lynch]

adding myself to the CC

Comment 11

[rusty.lynch]

Attachment: SanePlugin.tar.gz <-- Linux plugin for scanner/digital camera access

Comment 12

[rusty.lynch]

I've attached a tarball containing the SANE Linux plug-in.  This plug-in
allows scanner/digital camera access on a Linux system with the SANE
(Scanner Access Now Easy) libraries installed.

I would like to put this in mozilla/modules/plugin/SanePlugin as a sample
Linux plugin.

Comment 13

[Hoa Nguyen]

Has anyone had a chance to review the attached code yet?

Comment 14

[ekrock's old account (dead)]

Rusty, Hoa -- Many thanks! Sorry for the delay, Andrei & I have been swamped.

Andrei -- Since you're swamped & we all trust Intel's work on Linux plug-ins 
(any Linux plug-in sample code good enough for Intel will surely be useful to 
plug-in developers), would it be OK to (a) have them just check it in, and (b) 
have me publicize its availability to plug-in vendors? And is the location they 
propose to check it in the one you want?

Linux plug-in vendors could try the sample code out and then file any 
bugs/enhancement requests against it (using the Plugins component in Bugzilla) 
for tracking. Of course, since it's sample code from a third party, no one is 
necessarily providing a guarantee of long-term maintenance.

Comment 15

[leger]

Putting [nsbeta2+][6/15] radar.

Comment 16

[av (gone)]

I think this is fine. We can add it later to the SDK.

Comment 17

[ekrock's old account (dead)]

Great! Rusty, Hoa--thanks for the great work & please check it in!

Feel free to announce your contribution on netscape.public.mozilla.plugins and 
netscape.public.dev.plugin-upgrade. If for any reason you prefer not to announce 
it yourself, let me know & I'll take care of it; in that case let me know if you 
want to be anonymous for some reason as my default will be to thank you 
profusely for this great work!

Comment 18

[rusty.lynch]

I checked in the new plug-in.  I'll go ahead and send off a note to the
plugin newsgroup.

Comment 19

[rusty.lynch]

How about the patch I attached for the npsimple plugin.  Can I check that it 
too?

Comment 20

[ekrock's old account (dead)]

I'm not sure what patch you're referring to. If this is just more sample plug-in 
code for Linux (another sample? a fix/enhancement to the existing sample?) 
please go ahead. Thanks!

Comment 21

[rusty.lynch]

I was just talking about the patch that put a gtk text label in the 
plugin window of npsimple so that changing the 'text' property of the 
plugin was visible in the plug-in window (just like on the Win version.)

The code is checked in.

Comment 22

[av (gone)]

Should we close it now? Doing so.

Comment 23

[shrirang khanzode]

Marking verified since sample plugin has been checked in by Rusty.