Created attachment 310890 [details] xpi I try to upload When I try to a upload a new version of text/plain (https://addons.mozilla.org/en-US/firefox/addon/1864), I always get the following error: "No install.rdf present. Please see this page for reference." The xpi I try to upload is attached.
I'm hitting the same problem with one of my extensions (Thumbs).
Laura, do you think this might be another PHP5 bug that we are hitting up against? Assigning to fligtar for investigation...
Assignee: nobody → fligtar
Thought we tested this?
Created attachment 310906 [details] [diff] [review] Patch to fix logic This is an old logic error, which I don't think got invoked before because the RDF parser had a bug under PHP4. This patch should give you a more informative error message (the RDF parser can't read your install.rdf since it has unicode characters in it.) I think we *should* be able to parse them, so we'll have to hack on the RDF parser next. Leaving that for tomorrow.
Attachment #310906 - Attachment mime type: application/octet-stream → text/plain
Comment on attachment 310906 [details] [diff] [review] Patch to fix logic wfm; nice catch Laura
Attachment #310906 - Flags: review?(morgamic) → review+
Fixed in 3.1 in r11438.
OS: Windows XP → All
Hardware: PC → All
Target Milestone: --- → 3.1
Merged to trunk in r11440.
Seems to be working now on AMO.
Did this get pushed? If so I'll close it. If not I guess 3.2 will be pushed soon anyway.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Still happens to me even though this bug is marked "fixed"
This is a pretty old bug, so you should file a new one. However, I believe you already sent the file to us, so please be patient while we look into the problem.
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.