877973 - Some docs are still referring to 'packages' folder

Status: RESOLVED FIXED

(Add-on SDK Graveyard :: Documentation, defect, P2)

Description

[Matteo Ferretti [:zer0] [:matteo]]

Since the new layout of addon-sdk, we don't have "packages" anymore. We should update our documentation. For example:

https://addons.mozilla.org/en-US/developers/docs/sdk/latest/dev-guide/tutorials/adding-menus.html#Installing%20menuitems

Comment 1

[Will Bamberg [:wbamberg]]

What's the recommended way to use third-party modules now? Copy them under your lib directory?

Comment 2

[Matteo Ferretti [:zer0] [:matteo]]

I personally use a different approach – also before – so I'm not the right person to ask I guess: I use an external folder and pass the path to the `cfx`. Definitely if you copy them in the add-on lib folder they works. But it's better asking to Irakli, that redesign the SDK layout and loader.

However my main concern is remove the `packages` mention of the SDK folder from all the docs, that doesn't exists anymore: a lot of users asked in IRC because they were confused.

Comment 3

[Will Bamberg [:wbamberg]]

Attachment: a pull request

For the time being I've updated the tutorial to cover only the method where you create "packages" in your add-on. I understand that this is an interim fix and we'll need to update it when we have a new story for third-party modules, but I've verified that it works at least, which is more than the current tutorial does.

Comment 4

[Matteo Ferretti [:zer0] [:matteo]]

Comment on attachment 763681 [details]
a pull request

As we discussed in IRC, we all agreed to land this patch as documentation fix, and thinking about a proper solution for dependencies later.