Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 11 #17

Merged
merged 2 commits into from
Jun 11, 2012
Merged

Issue 11 #17

merged 2 commits into from
Jun 11, 2012

Conversation

erikvold
Copy link
Collaborator

No description provided.

brianking added a commit that referenced this pull request Jun 11, 2012
@brianking brianking merged commit 8b41d0e into brianking:master Jun 11, 2012
@brianking
Copy link
Owner

protocol is not resolving:

$ cfx run
Traceback (most recent call last):
File "c:/mozilla/addon-sdk/addon-sdk-1.7/bin/cfx", line 33, in
cuddlefish.run()
File "c:\mozilla\addon-sdk\addon-sdk-1.7\python-lib\cuddlefish__init__.py", l
ine 626, in run
deps = packaging.get_deps_for_targets(pkg_cfg, targets)
File "c:\mozilla\addon-sdk\addon-sdk-1.7\python-lib\cuddlefish\packaging.py",
line 275, in get_deps_for_targets
raise PackageNotFoundError(dep, required_reason)
cuddlefish.packaging.PackageNotFoundError: protocol (required by 'remo-companion
')
(addon-sdk-1.7)

@erikvold
Copy link
Collaborator Author

this guide explains a little about how to use 3rd party modules

protocol is here. I usually clone the repo to ~/github/ then ln -s ~/github/protocol protocol from the packages folder of my sdk clone.

@brianking
Copy link
Owner

Oh I see. So does that do a copy, and will 'cfx xpi' then bundle it?

Also, that will be flagged by the AMO validator I am sure as not being a core SDK file.

@erikvold
Copy link
Collaborator Author

yes and yes.

@brianking
Copy link
Owner

Thanks, works now.

This checkin has caused 1 regression. I have filed issue #18 for it.

@brianking
Copy link
Owner

Can about: pages have a favicon I wonder?

@erikvold
Copy link
Collaborator Author

I just tried adding a favicon with <link rel="icon" type="image/jpg" href="..."> and that worked.

@brianking
Copy link
Owner

Seems like protocol is broken in 1.8:

Gozala/jetpack-protocol#5

@erikvold
Copy link
Collaborator Author

hmm it looks like this lib is going to be part of the sdk mozilla/addon-sdk#290

We may still need to work around the 1.8 issue tho..

I'll ask @Gozala about it.

@Gozala
Copy link

Gozala commented Jul 16, 2012

If someone want's to send in pull request I'd be happy to accept. The problem is module wrapper here: https://github.com/Gozala/jetpack-protocol/blob/master/selfish.js#L6

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.

None yet

3 participants