D6 version update

I just committed a first Drupal 6 version to HEAD. Please try it out.

http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/site_map/

subscribe
and agreed about creating a HEAD 6.x release.

subscribe, but dev, not head, surely?

HEAD: "there can be only one"

If the HEAD branch is the one that is 6.x compatible then it's perfectly reasonable to create the 6.x-1.x release from that. You can then tag it with DRUPAL-6--1-0 and create the 6.x-1.0 from that. Etc. Until you need to use HEAD for something else at which point you can branch it, change the 6.x-1.x release to point to the branch, and create a new HEAD release for whatever other purpose.

See http://drupal.org/node/17570#HEAD for more details on this idea and other strategies.

Bottom line - a release of any kind would help whether on the HEAD or on a DRUPAL-6--1 branch.

Seems to work OK for me, apart from a small bug in line 84 of site_map.module, complaining of too few arguments:

Before:
$block['content'] = theme('feed_icon', url($feedurl));

After small modification:
$block['content'] = theme('feed_icon', url($feedurl), "Syndicate Content");

btw, I checked out using the following command (I assume this is correct to get the latest version you refer to):

cvs -z6 -d:pserver:anoymous:anoymous@cvs.drupal.org/cvs/drupal-contrib checkout contributions/modules/site_map

Also, would it be possible to adapt this to provide an xml sitemap file to be queried by search engines?

Thanks,
Dan

@dmoore - there is already the xmlsitemap project which creates an xml sitemap for search engines. We shouldn't duplicate that functionality in this module.

Thanks.

It does seem there is much overlap between these two projects and would be useful to have the same level of configurability in both. The main difference seems to be the style of output. No big deal though - both have their place.

Regards,
Dan

These two projects are in no way related, aside from the fact that they provide a type of site map, and the module names both contain the words: "site" and "map".

?

Now there is a 6-dev version and I have committed the fix to theme_feed_icon(), thanks dmoore.