Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#15376 closed task (fixed)

CometD split out from DojoX

Reported by: dylan Owned by: dylan
Priority: blocker Milestone: 1.8
Component: DojoX Cometd Version: 1.7.2
Keywords: Cc:
Blocked By: Blocking:

Description

CometD 2.5 beta 1 will be released on Monday. As part of that effort, they have refactored their client to be AMD-based and work well with 1.7.

As such, we should remove the code that is in DojoX, and setup an svn:externals to their 2.5 GitHub? tree.

They will also be submitting their package to cpm/Dojo Foundation Package Manager.

I consider this to be blocking the 1.8 release, as dojox/cometd has been unnecessarily crufty for quite some time.

Change History (7)

comment:1 Changed 6 years ago by dylan

http://bugs.cometd.org/browse/COMETD-336

Though they are using dojo.registerModulePath... that doesn't seem at all right for Dojo 1.7+. We need to do a review and offer some assistance.

comment:2 Changed 6 years ago by dylan

Actually, digging deeper, they support both that for people using dojo.require, or package map configuration for AMD users.

So it looks like we need to test and then setup the svn:externals.

comment:3 Changed 6 years ago by dylan

CometD has an updated module we could use, but they're struggling to understand how to make an unbuilt module available for the Dojo build system to use.

comment:4 Changed 6 years ago by bill

If you are going to remove dojox/cometd and add an svn:external (or even if you aren't adding the svn:external), please do it before we cut the beta, i.e. before Wednesday (June 13, 2012).

comment:5 Changed 6 years ago by dylan

In [28838]:

refs #15376, step 1 of switching cometd client to their project

comment:6 Changed 6 years ago by dylan

Resolution: fixed
Status: newclosed

This is pretty much done, just need to keep working with the CometD team to get their GitHub? repo setup optimally for Dojo users.

comment:7 Changed 6 years ago by bill

Note: See TracTickets for help on using tickets.