Bug 585467 (googlesharing)

*** Bug 585466 has been marked as a duplicate of this bug. ***

Minor changes

Added missing images directory

Spec URL:
http://github.com/downloads/matriux/fedora/mozilla-googlesharing.spec-v2 (renamed)
SRPM URL:
http://github.com/downloads/matriux/fedora/mozilla-googlesharing-0.18-2.fc11.src.rpm

I'm no sponsor, but here are a few comments:

- You install it a bit strange, and I didn't expect it to build cleanly because of the 'cd -'...

  It would be better, if you use pushd and popd like:

pushd chrome/content/
install -Dpm 644 addEditProxy.js googlesharing.js options.js addEditProxy.xul googlesharing.xul options.css options.xul %{buildroot}%{inst_dir}/chrome/content/
popd

With pushd you go into the folder and with popd to the folder, you came from.
Looks nicer and should be a bit more sane.

- Please overwrite the spec file on githup and don't add something like '-v2'. Old versions will be available throught olg src.rpms.
If you want to also keep the old versions, please make a subfolder for each version or something like that.

- What about seamonkey? Does this programm works with that too?
  There are some additional folders needed for that and *maybe* that's it.

  Here is an example in mozilla-noscript, which works well:
  http://cvs.fedoraproject.org/viewvc/devel/mozilla-noscript/mozilla-noscript.spec?view=markup

- There won't be a debuginfo package, because you have BuildArch: noarch, so you can delete:
  %define debug_package %{nil}

- $ rpmlint mozilla-googlesharing-0.18-2.fc13.src.rpm noarch/mozilla-googlesharing-0.18-2.fc13.noarch.rpm 
mozilla-googlesharing.src: W: spelling-error Summary(en_US) Anonymizing -> Anonymity, Anatomizing, Anonymous
mozilla-googlesharing.src: W: spelling-error Summary(en_US) google -> Google, goggle, googly
mozilla-googlesharing.src: W: spelling-error %description -l en_US addon -> ad don, ad-don, add on
mozilla-googlesharing.src: W: spelling-error %description -l en_US google -> Google, goggle, googly
mozilla-googlesharing.src: W: spelling-error %description -l en_US unredirected -> redirection, misdirected, unrestricted
mozilla-googlesharing.noarch: W: spelling-error Summary(en_US) Anonymizing -> Anonymity, Anatomizing, Anonymous
mozilla-googlesharing.noarch: W: spelling-error Summary(en_US) google -> Google, goggle, googly
mozilla-googlesharing.noarch: W: spelling-error %description -l en_US addon -> ad don, ad-don, add on
mozilla-googlesharing.noarch: W: spelling-error %description -l en_US google -> Google, goggle, googly
mozilla-googlesharing.noarch: W: spelling-error %description -l en_US unredirected -> redirection, misdirected, unrestricted
2 packages and 0 specfiles checked; 0 errors, 10 warnings.

Using Google would reduce 4 warnings, the rest seems unavoidable...

Spec URL : http://github.com/downloads/matriux/fedora/mozilla-googlesharing.spec
SRPM URL : http://github.com/downloads/matriux/fedora/mozilla-googlesharing-0.18-3.fc11.src.rpm

-Used pushd and popd
-Fixed typo google
-Removed %define debug_package %{nil} from spec file

1) use %global over %define. See https://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_over_.25define

2) you should take reference from http://cvs.fedoraproject.org/viewvc/devel/mozilla-noscript/mozilla-noscript.spec?view=markup

you should write  "Requires:       mozilla-filesystem" and not BuildRequires.

3) If you want to follow macro style usage then write macro for rm command also.
Complete macro listing you can find on your system in file 
/usr/lib/rpm/macros

Updated Spec and Src Rpm

SPEC URL:http://github.com/downloads/matriux/fedora/mozilla-googlesharing.spec
SRPMS URL:http://github.com/downloads/matriux/fedora/mozilla-googlesharing-0.18-4.fc11.src.rpm

(In reply to comment #4)
> -Used pushd and popd
> -Fixed typo google
> -Removed %define debug_package %{nil} from spec file    

So you are not interessted to make this programm starting for seamonkey users?

Thomas, i will update the spec soon.

Updated 
SPEC:http://github.com/downloads/matriux/fedora/mozilla-googlesharing.spec
SRPM:http://github.com/downloads/matriux/fedora/mozilla-googlesharing-0.18-5.fc11.src.rpm

install.rdf does not mention any id for seamonkey installation. So I don't know if there is any other way to make this extension work in seamonkey.

Thomas,
   Can you help here to get this extension working in seamonkey as per you asked in your comment#3?

Otherwise I see this package is ready for Final review.

I see you have done good reviews. So I am going to sponsor you now and therefore removing FE-NEEDSPONSOR.

(In reply to comment #10)
> install.rdf does not mention any id for seamonkey installation. So I don't know
> if there is any other way to make this extension work in seamonkey.
> 
> Thomas,
>    Can you help here to get this extension working in seamonkey as per you
> asked in your comment#3?
> 
> Otherwise I see this package is ready for Final review.    

Here is what I did for mozilla-noscript:
1 	%global moz_extensions %{_datadir}/mozilla/extensions
2 	
3 	%global firefox_app_id \{ec8030f7-c20a-464f-9b0e-13a3a9e97384\}
4 	%global src_ext_id \{73a6fe31-595d-460b-a920-fcc0f8843232\}
5 	%global inst_dir %{moz_extensions}/%{firefox_app_id}/%{src_ext_id}
6 	
7 	%global seamonkey_app_id \{92650c4d-4b8e-4d2a-b7eb-24ecf4f6b63a}
8 	%global sym_link_dir %{moz_extensions}/%{seamonkey_app_id} 

I found this ids e.g. at https://wiki.mozilla.org/Extension_Manager:Addon_Update_Security:Signature

There are more ids, than just seamonkey and firefox, but at my initial review, I was also asked to add seamonkey, because that's also in fedora...

After looking at the spec, this should be done, when you replace the value of seamonkey_app_dir to the seamonkey_app_id from above.

Just tested that with seamonkey (not with firefox yet):

--- ./orig	2010-05-04 13:17:56.714392098 +0200
+++ ./mozilla-googlesharing.spec	2010-05-04 13:18:25.557395025 +0200
@@ -4,7 +4,7 @@
 %global src_ext_id googlesharing.org
 %global inst_dir %{moz_extensions}/%{firefox_app_id}/%{src_ext_id}
 
-%global seamonkey_app_dir \{googlesharing}
+%global seamonkey_app_dir \{92650c4d-4b8e-4d2a-b7eb-24ecf4f6b63a}
 %global sym_link_dir %{moz_extensions}/%{seamonkey_app_dir} 
 
 Name:           mozilla-googlesharing


Works with seamonkey (at least it's showing up in 'installed add-ons' didn't test if the programm actually works, but guessing so).

i checked on firefox,works fine and it seems googlesharing doesnt work on seamonkey 1.1.19.
ok,i will update the spec file tomorrow.
thanks

(In reply to comment #14)
>it seems googlesharing doesnt work on seamonkey 1.1.19.

FYI: I tested it with: seamonkey-2.0.4-1.fc13.x86_64

I tried above patch on my F-13 machine and see its showing in installed extensions but also saw showing "Not compatible with SeaMonkey 2.0.4". So If its not going to work in SeaMonkey then better not to include it in SPEC.

(In reply to comment #16)
> I tried above patch on my F-13 machine and see its showing in installed
> extensions but also saw showing "Not compatible with SeaMonkey 2.0.4". So If
> its not going to work in SeaMonkey then better not to include it in SPEC.    

Oh, right...
(As noted above not tested, just it's showing up^^)

Maybe it's worth to suround a conditional around the seamonkey part, so it can easily enabled/disabled. Don't know if it will ever be working with seamonkey (I don't use seamonkey anyway, just wanted to note, that some user want to have that part...)

--- mozilla-googlesharing.spec	2010-05-06 18:09:50.058954346 +0200
+++ orig	2010-05-04 13:17:56.714392098 +0200
@@ -4,11 +4,9 @@
 %global src_ext_id googlesharing.org
 %global inst_dir %{moz_extensions}/%{firefox_app_id}/%{src_ext_id}
 
-%global seamonkey_app_dir \{92650c4d-4b8e-4d2a-b7eb-24ecf4f6b63a}
+%global seamonkey_app_dir \{googlesharing}
 %global sym_link_dir %{moz_extensions}/%{seamonkey_app_dir} 
 
-%global with_seamonkey 0
-
 Name:           mozilla-googlesharing
 Version:        0.18
 Release:        5%{?dist}
@@ -60,10 +58,8 @@
 popd
 
 # symlink from seamonkey extension to firefox extension
-%if 0%{?with_seamonkey}
 mkdir -p %{buildroot}%{sym_link_dir}
 ln -s %{inst_dir} %{buildroot}%{sym_link_dir}
-%endif
 
 %clean
 %{__rm} -rf $RPM_BUILD_ROOT
@@ -72,9 +68,7 @@
 %defattr(-,root,root,-)
 %doc COPYING
 %{inst_dir}
-%if 0%{?with_seamonkey}
 %{sym_link_dir}
-%endif
 
 
 %changelog

I agree with Parag.Since its not compatible with seamonkey,better not to include in spec.Im reverting back changes made in comment 9.

Spec URL:http://github.com/downloads/matriux/fedora/mozilla-googlesharing.spec
SRPM :http://github.com/downloads/matriux/fedora/mozilla-googlesharing-0.18-4.fc11.src.rpm

Review:
+ package builds in mock (rawhide i686).
koji Build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=2176309
+ rpmlint output for SRPM and for RPM.
mozilla-googlesharing.src: W: spelling-error Summary(en_US) Anonymizing -> Anonymity, Anatomizing, Anonymous
mozilla-googlesharing.src: W: spelling-error %description -l en_US google -> Google, goggle, googly
mozilla-googlesharing.noarch: W: spelling-error Summary(en_US) Anonymizing -> Anonymity, Anatomizing, Anonymous
mozilla-googlesharing.noarch: W: spelling-error %description -l en_US google -> Google, goggle, googly
==> Ignore this.
+ source files match upstream url (sha1sum)
af55ae4f48b2eb02eef0116e3269fa9898f02407  googlesharing-0.18-fx.xpi

+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ mozilla firefox extension.

APPROVED.

New Package CVS Request
=======================
Package Name: mozilla-googlesharing
Short Description: Anonymizing proxy service for google sharing system
Owners: matriux
Branches: F-11 F-12
InitialCC:

Do you really not want an F-13 branch?

Thanks for bringing to my notice :)

New Package CVS Request
=======================
Package Name: mozilla-googlesharing
Short Description: Anonymizing proxy service for google sharing system
Owners: matriux
Branches: F-11 F-12 F13 
InitialCC:

CVS Done

I think this package is already built for requested branches. Any reason for not closing this review?

Hi Parag,

    Closing the review,thanks

The project's website does not write anything about it: http://www.googlesharing.net/.
The source of the code is not located: http://releases.mozilla.org/pub/mozilla.org/addons/60333/googlesharing-0.18-fx.xpi.
XUL development has been discontinued.
It seems that you need to remove this package from the repository.