search

infinity0 / mozilla-gnome-keyring-legacy

A firefox extension that enables Gnome Keyring integration (legacy version)
https://bugzilla.mozilla.org/show_bug.cgi?id=309807
Other
55 stars 8 forks source link

does not work with iceweasel 24.0 #40

Closed mimbert closed 10 years ago

mimbert commented 10 years ago

Hi,

I tryed compiling mozilla-gnome-keyring (version tagged 0.6.9) for iceweasel 24.0 (currently in debian unstable).

I forced XUL_VER_MAX = 24.* in the Makefile. It compiles ok and installs ok in iceweasel (iceweasel refuses to install it for version incompatibility if I don't force XUL_VER_MAX = 24.*).

But then it does not work, all passwords stored are gone.

infinity0 commented 10 years ago

Did you compile against xulrunner-dev 24.0? To force this, you must change the appropriate line in debian/control.

I'd assume your passwords are not "gone" - you can probably still access them by running seahorse.

mimbert commented 10 years ago

I assumed i compiled again xulrunner-dev 24.0 because when I tried upgrading to iceweasel 24, it also upgraded to xulrunner 24.0. But I now notice that it did not upgraded xulrunner-dev to 24.0 as well, so I had xulrunner 24 together with xulrunner-dev 17. Could it be the cause of the issue?

By the way, i don't use debian/control since I compile with the Makefile, not using the debian build process (I do not try to build a debian package, I only need to build the xpi)

And yes of course my passwords were still there, but they were not prefilled in iceweasel anymore.

Should I try to recompile after upgrading xulrunner-dev to 24?

infinity0 commented 10 years ago

Yes, try to recompile after that upgrade. The -dev package is the important one that you need to build against, otherwise it's known that it won't work. I'll try to make this a bit more explicit in the instructions next time I have a look at this package.

mimbert commented 10 years ago

I upgraded xulrunner-dev to 24, recompiled mozilla-gnome-keyring, reinstalled it and everything is fine, it works. thanks for the help