Error: The system was unable to find the specified registry key or value

RESOLVED FIXED

Status

Add-on SDK
General
P3
normal
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: Erik Vold, Assigned: myk)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [cherry-pick-1.3])

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

7 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.4) Gecko/20100611 Firefox/3.6.4
Build Identifier: 

I get the following error when I bin\activate:

"Error:  The system was unable to find the specified registry key or value"

everything else seems to be working ok so far..

Reproducible: Always

Steps to Reproduce:
0. On windows XP..
1. hg clone http://hg.mozilla.org/labs/jetpack-sdk/
2. cd jetpack-sdk
3. bin\activate
Actual Results:  
C:\jetpack-sdk\jetpack-sdk>bin\activate

Error:  The system was unable to find the specified registry key or value
Welcome to the Jetpack SDK. Run 'cfx docs' for assistance.

Expected Results:  
C:\jetpack-sdk\jetpack-sdk>bin\activate

Welcome to the Jetpack SDK. Run 'cfx docs' for assistance.

Comment 1

7 years ago
I think this was a problem with python 2.7 as well. Try going back to 2.6.
(Assignee)

Updated

7 years ago
Duplicate of this bug: 588245
(Assignee)

Comment 3

7 years ago
I see the problem, and I'm using Python 2.6.
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Comment 4

7 years ago
The Add-on SDK is no longer a Mozilla Labs experiment and has become a big enough project to warrant its own Bugzilla product, so the "Add-on SDK" product has been created for it, and I am moving its bugs to that product.

To filter bugmail related to this change, filter on the word "looptid".
Component: Jetpack SDK → General
Product: Mozilla Labs → Add-on SDK
QA Contact: jetpack-sdk → general
(Assignee)

Updated

7 years ago
Assignee: nobody → myk
(Assignee)

Updated

7 years ago
Priority: -- → P2
Target Milestone: --- → 1.0
(Assignee)

Comment 5

6 years ago
(automatic reprioritization of 1.0 bugs)
Priority: P2 → P1
Target Milestone: 1.0 → 1.1
(Assignee)

Updated

6 years ago
Priority: P1 → P3
Target Milestone: 1.1 → 1.2
(Pushing all open bugs to the --- milestone for the new triage system)
Target Milestone: 1.2 → ---
This is possibly related to bug 684136. I no longer see this error after applying the pull request in that bug (and removing all Python registry entries and reinstalling Python).

Anyone care to confirm?
(Assignee)

Comment 8

6 years ago
Created attachment 566982 [details] [diff] [review]
patch v1: fixes problem

(In reply to Wes Kocher (:KWierso) (Jetpack Bugmaster) from comment #7)
> This is possibly related to bug 684136. I no longer see this error after
> applying the pull request in that bug (and removing all Python registry
> entries and reinstalling Python).
> 
> Anyone care to confirm?

I noticed that too, but it turns out that change just masked the bug, which is a combination of:

1. the `REG QUERY "%PYTHONKEY%\%PYTHONVERSION%\InstallPath" /ve` call doesn't redirect stdout/err to /dev/null, so any errors generated by that command are printed to the terminal (unlike other such calls in the script, which are all redirected);

2. the `if %ERRORLEVEL% EQU 0` checks inside the `if` conditional blocks are not checking the return codes they think they're checking, because %ERRORLEVEL% is expanded before the conditional block is evaluated, per <http://stackoverflow.com/questions/4367930/errorlevel-inside-if> and <http://batcheero.blogspot.com/2007/06/how-to-enabledelayedexpansion.html>.

The fixes are to redirect the output of that command to /dev/null and enable delayed expansion of environment variables.  Here's a patch that does that.  There are other places where %ERRORLEVEL% is checked outside of conditionals; not sure if those should be changed to !ERRORLEVEL! as well (or, alternately, `IF ERRORLEVEL`, which doesn't have this problem).

Mark: I don't know if this matters given plans to land some version of the batch file in bug 571483, but I figured this stuff out in the process of reviewing the fix for bug 684136, so I figured it was worth at least rolling the patch.
Attachment #566982 - Flags: feedback?(mhammond)
Comment on attachment 566982 [details] [diff] [review]
patch v1: fixes problem

I haven't tested it, but it looks good to me.  Note however that this patch will be made obsolete if the one in bug 571843 lands.
Attachment #566982 - Flags: feedback?(mhammond) → feedback+
(Assignee)

Comment 10

6 years ago
Created attachment 571369 [details] [diff] [review]
patch v2: resolves trivial application conflicts

Given the lack of progress on bug 571843, and in order to capture the learnings from this bug in the historical record, let's move forward on reviewing and landing this bug.  This patch applies to the tip, resolving trivial application conflicts from recent changes.
Attachment #566982 - Attachment is obsolete: true
Attachment #571369 - Flags: review?(warner-bugzilla)
Comment on attachment 571369 [details] [diff] [review]
patch v2: resolves trivial application conflicts

I have no idea what this patch does: my windows-fu is non-existent. If it works for you, though, I see no reason to not apply it.
Attachment #571369 - Flags: review?(warner-bugzilla) → review+
(Assignee)

Comment 12

6 years ago
https://github.com/mozilla/addon-sdk/commit/ddace880c0b18b2c7383a814a176802a18e7c45c
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Comment 13

6 years ago
https://github.com/mozilla/addon-sdk/commit/d245ef8e91275161b245e3f8649448303cb78508
Whiteboard: [cherry-pick-1.3]

Comment 14

6 years ago
Commit pushed to https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/d245ef8e91275161b245e3f8649448303cb78508
bug 574563: correctly check ERRORLEVEL in activate.bat; squelch REG QUERY error; f=@mhammond, r=@warner
You need to log in before you can comment on or make changes to this bug.