Skip to content

Update for Thunderbird 68.0#10

Closed
bit wants to merge 1 commit intodennisverspuij:masterfrom
bit:tb68
Closed

Update for Thunderbird 68.0#10
bit wants to merge 1 commit intodennisverspuij:masterfrom
bit:tb68

Conversation

@bit
Copy link
Copy Markdown

@bit bit commented Aug 30, 2019

No description provided.

@bit bit mentioned this pull request Aug 30, 2019
@designchris
Copy link
Copy Markdown

Does anyone know what needs to be done to support TB68?

@leres
Copy link
Copy Markdown
Contributor

leres commented Sep 5, 2019

I wrote a python script to convert install.rdf files to manifest.json:

[https://github.com/leres/mozilla-install-rdf-to-manifest-json](https://github.com/leres/mozilla-install-rdf-to-manifest-json)

And the manifest.json I've been using for the last few weeks is a little different from the one in pull request #10.

@designchris
Copy link
Copy Markdown

@leres Thanks, I tried to run your script but it's throwing an error:

File "./convert-rdf.py", line 22
print(*args, file=sys.stderr)
^
SyntaxError: invalid syntax

@leres
Copy link
Copy Markdown
Contributor

leres commented Sep 9, 2019

@leres Thanks, I tried to run your script but it's throwing an error:

File "./convert-rdf.py", line 22
print(*args, file=sys.stderr)
^
SyntaxError: invalid syntax

I just made a change to add python 2 compatibly, could you please get the new version and try again?

@designchris
Copy link
Copy Markdown

Thank you! At first it failed on Windows, but it works great on Linux!

@htgoebel
Copy link
Copy Markdown

I confirm this pull-request makes tb-correctidentity work on Thunderbird 68.0

@pbonbon
Copy link
Copy Markdown

pbonbon commented Oct 2, 2019

Plugin doesn't work correctly on Windows 10 using Thunderbird 68.1.1. The configuration cannot be saved.

@peci1
Copy link
Copy Markdown

peci1 commented Nov 2, 2019

See bit#1 . I got the fix there and let's hope it will propagate to this PR, too.

@pbonbon
Copy link
Copy Markdown

pbonbon commented Nov 5, 2019

See bit#1 . I got the fix there and let's hope it will propagate to this PR, too.

It works! Thank you very much!

@bertvandepoel
Copy link
Copy Markdown

@bit I just pulled from your repo and zipped the tb68 branch, then zipped the result. Weirdly, Thunderbird keeps on saying that it's incompatible with 68.4.1. Any guess what I'm doing wrong? Thx :)

@SimonMaenaut
Copy link
Copy Markdown

Sadly the following error appears when I try to install this extension in Thunderbird 68.5.0 on Linux.
addons.xpi WARN Invalid XPI: Error: File /home/simon/Downloads/tb-correctidentity-tb68.zip does not contain a valid manifest(resource://gre/modules/addons/XPIInstall.jsm:671:11) JS Stack trace: loadManifest@XPIInstall.jsm:671:11

@SimonMaenaut
Copy link
Copy Markdown

Zipping the individual elements together instead of a directory with everything inside seems to do the trick :-)

@bertvandepoel
Copy link
Copy Markdown

It seems to install fine with the instructions by @SimonMaenaut but the addon doesn't work for me at that point. I then went on to explore and found that https://github.com/rwg0/tb-correctidentity/tree/tb68 by @rwg0 had a fix for problems with fetching identities (which I seemed to have) and also merged in another fix by @peci1 for saving of settings.
Could you contemplate merging those, @bit , and then adding them to this PR? Hopefully that will convince @dennisverspuij to merge this PR and release on AMO.

@ldreier ldreier closed this Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants