i tried using the latest version of dhtml_menu from the DRUPAL-5 branch. turns out this version doesn't work with Drupal 5.x (this commit breaks it - note the changes to dhtml_menu()). looking at the CVS changelogs, i also noted that there is a DRUPAL-6--1-0 tag on the DRUPAL-5 branch. this is clearly wrong. the above mentioned commit should have gone to either HEAD or a new DRUPAL-6--1 Drupal 6.x compatible development branch. now we have Drupal 6 development going on a) on a wrong DRUPAL-5 branch, b) parallel on HEAD, and c) no branch for Drupal 5 fixes and updates.

to remedy this, i created a new DRUPAL-6--1 branch based on latest HEAD and merged in the changes that went into the wrong DRUPAL-5. on the way, i fixed some doc issues to be more in line with Drupal standards (@file, order of comments, authors in README.txt to avoid duplication). this version is tested and working with Drupal HEAD of today.

TODO:
* remove the Drupal 6.x api changes from the DRUPAL-5 branch
* merge fixes from HEAD into this
* (re)move DRUPAL-6--1 tag
* possibly make new releases

please update your cvs checkouts and work on the proper branches (DRUPAL-5 for Drupal 5.x development - once TODO #1 is done, DRUPAL-6--1 for Drupal 6.x development).

see also CVS maintainer quick-start guide

Comments

ax’s picture

ax commented

* remove the Drupal 6.x api changes from the DRUPAL-5 branch
* merge fixes from HEAD into this

are DONE, see http://drupal.org/cvs?commit=97095 .

* (re)move DRUPAL-6--1 tag

was a typo - it should be "(re)move DRUPAL-6--1-0 tag". leaving this for someone else.

ax’s picture

ax commented
Status: Active » Fixed

changed the CVS branch of the dhtml_menu 6.x-1.x-dev release from HEAD to DRUPAL-6--1.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.