This is an FYI for everyone. I'm going to be merging the 6.x-2.x branch into the HEAD CVS branch so that I stop getting the CVS attic errors when I commit. Once that's done I'll reassign the CVS branch for the 6.x-2.x development build. From there, once we're ready to port 6.x-2.x to Drupal 7, I we re-split the code again.

Comments

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented

Why don't I get any error, when I commit to the branch I am developing?

Anonymous’s picture

Anonymous (not verified) commented

I've never seen an attic error either. Are you sure it isn't your local working copy that isn't in error?

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Mosted editors / CVS command line work around it just fine, but apparently Komodo IDE causes big problems. I'll look into filing a bug with my editor. See also http://www.cs.utah.edu/dept/old/texinfo/cvs/cvs_12.html. Plus, this makes sense if the 6.x-2.x branch is the one that will be brought forward/ported to Drupal 7.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented

I am using Komodo IDE too, but I have never had problems with any files in attic.

To create a Drupal 7 version should be enough to create a Drupal 7 branch; I don't see any reason to merge the code for Drupal 6 with HEAD.

Anonymous’s picture

Anonymous (not verified) commented

I don't see any reason to merge the code for Drupal 6 with HEAD.

What Dave suggests is what is supposed to happen before creating the next major branch. The next major branch should always be created from HEAD. You can only do that if the changes are merged into HEAD.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented

The next major branch should always be created from HEAD.

That is true, but (so far) he has not gotten from Darren Oh the permission to make a Drupal 7 version. Even if Darren Oh has not written code for XML sitemap in the past months, the maintainer is still him.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Postponed

As per my conversation with Darren, we're going to wait until the module is ready to port to Drupal 7 to merge the CVS branches. Marking as postponed.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Postponed » Active

Once I get an alpha release (next couple of days), I want to do this CVS merging so I can start working on porting the module to D7. Please put in any objections now.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Merge completed and HEAD is now the official D7 branch. Now all the times that I'm thinking "Ah, I should really do this in D7" I can actually put it in code.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Fixed

Dev build will be at http://drupal.org/node/190851

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Wow, I should have done this way sooner. This is going to be a lot of work to port to Drupal 7. Should have been doing this the entire time I was working on 6.x-2.x.

:/

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Drupal 7 porting started and in minimal-working state: #666382: XML sitemap Drupal 7 port

Anonymous’s picture

Anonymous (not verified) commented

Good to hear.

Status: Fixed » Closed (fixed)

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