Alexandre Poirot [:ochameau] Assignee
	Description • 9 years ago

Let's make a dedicated bug to get rid of dbg-server.jsm.
This jsm is useless as we should import this module via
   const { DebuggerServer } = require("devtools/server/main");
(Instead of Cu.import("resource://gre/modules/devtools/dbg-server.jsm", {}))

All usages of dbg-server.jsm prevent correctly reloading server files when using local source directory.

	Alexandre Poirot [:ochameau] Assignee
	Comment 1 • 9 years ago

Let's see what try says:
  https://treeherder.mozilla.org/#/jobs?repo=try&revision=8843ced5ad39

	Alexandre Poirot [:ochameau] Assignee
	Comment 2 • 9 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b7c7b7b7dd2c
This time, I'm not removing the jsm as it is used in too many addons.

	Alexandre Poirot [:ochameau] Assignee
	Comment 3 • 9 years ago

... but I've dispatched some previous try run with dbg-server.jsm removed to verify there was no more usages. Some new one may slit through, we would have to verify during reviews and may be a bit latter, check m-c codebase for possible new usages of it.

	J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)
	Comment 4 • 9 years ago

Comment on attachment 8644911 [details] [diff] [review]
patch v2

Review of attachment 8644911 [details] [diff] [review]:
-----------------------------------------------------------------

Should dbg-server.jsm log a deprecation notice when it's used?

	Pulsebot
	Comment 5 • 9 years ago

https://hg.mozilla.org/integration/fx-team/rev/2fcd73fa8a7e

	Alexandre Poirot [:ochameau] Assignee
	Comment 6 • 9 years ago

(In reply to J. Ryan Stinnett [:jryans] (use ni?) from comment #4)
> Comment on attachment 8644911 [details] [diff] [review]
> patch v2
> 
> Review of attachment 8644911 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Should dbg-server.jsm log a deprecation notice when it's used?

It's not a big deal if they keep using it or if we keep this alias.
But yes, we could do that. I would do that a bit later once file are moved, loader paths are cleaned up and everything is more stable as things might change again.

	Alexandre Poirot [:ochameau] Assignee
	Comment 7 • 9 years ago

Btw, I've pushed two try just before landing.
One removing dbg-server.jsm:
  https://treeherder.mozilla.org/#/jobs?repo=try&revision=9cc2dd728d33
And another without removing it (what actually landed):
  https://treeherder.mozilla.org/#/jobs?repo=try&revision=585e58baa292

Everything was green (enough)!

	[github robot]
	Comment 8 • 9 years ago

Commit pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/a50a0c78b557af3994fa78bbc8d830a3c77b6620
Bug 1190452 - Always import toolkit/devtools/server/main.js as CommonJS module. r=jryans

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 9 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/2fcd73fa8a7e