Since I own this module, guess where the most up-to-date code is?

Site Documentation HEAD (2008-Jan-17)    Update available
Recommended version: 	5.x-1.5 (2007-Nov-18)
Development version: 	5.x-1.x-dev (2008-Jan-17)

Comments

dww’s picture

Status: Active » Closed (won't fix)

If you had a release node pointing to HEAD, this wouldn't be a problem.
Feel free to use the settings to have update_status ignore this module if you prefer.

dww’s picture

Status: Closed (won't fix) » Closed (works as designed)

Actually, it's not that I "won't fix" this. Really, it's not a problem at all -- you can already solve this yourself in 1 of 2 ways.

nancydru’s picture

5.x-1.x-dev: Nightly development snapshot from CVS branch: HEAD

So this is more of my problems in creating "official" releases?

Actually maybe this gets more into a question I was going to ask you before I put it into my new docs. Do I need to take all the modules I created and set a sticky tag of "DRUPAL-5" (currently they have none) for the 5.x branches before I start my 6.x conversions. And before I commit anything for 6.x, have those versions with a sticky tag of "DRUPAL-6--1"? And, yes, I now have found out how to set the sticky tag with Tortoise - it's weird, but what they say to do.

dww’s picture

I'm assuming:

a) you have the latest cvs_deploy and update_status installed and working
b) your local copy of sitedoc is a regular CVS checkout from cvs.d.o
c) you "Check manually"'ed recently and have accurate available update data

if all that's true, i don't see how update_status could be telling you your locally installed version is just "HEAD". please either provide screenshots and more info, or confirm that the problem went away when you refreshed and/or updated everything.

in terms of your question -- no it's not the sticky tags that matter (those just affect your local workspace), it's creating branches (an operation in the repository itself). yes, you should make DRUPAL-5 branches before you start porting to D6 in HEAD. see http://drupal.org/node/17570

nancydru’s picture

StatusFileSize
new59.38 KB
new124.58 KB

I'm as sure as you are that I'm doing something wrong, but here are the answers.

a) yes - just downloaded because of today's posts.
b) no - it is the original source with CVS on top of it.
c) yes

Screenshots attached.

dww’s picture

Status: Closed (works as designed) » Postponed (maintainer needs more info)

I don't understand "it is the original source with CVS on top of it." What does that mean?

Can you post a directory listing and the contents of every file in the "CVS" folder in one of these directories, for example: c:\www\drupal\sites\all\modules\sitedoc\CVS

I'm assuming there's no "Tag" file in there, but there *is* a "Repository" file and a "Root" file. I'm curious what those files contain, and if those files in fact exist. I must admit, I've never tried to use CVS on a windoze machine, so it's entirely possible cvs_deploy is confused since it was written by and for *nix users. ;)

Also, please include the contents of sitedoc.info.

nancydru’s picture

StatusFileSize
new35.92 KB
new49.37 KB
new50.78 KB

Original source means that is where I developed the module. When I was ready to contribute it, I then told Tortoise to build a CVS repository for that directory.

sitedoc.info:

; $Id: sitedoc.info,v 1.5 2007/06/19 13:02:37 nancyw Exp $
name = Site Documentation
description = "Displays various pieces of information about this Drupal installation."

Entries:

/README.txt/1.1/Sat May 26 03:15:07 2007//
/sitedoc.info/1.5/Tue Jun 19 13:02:37 2007//
/sitedoc.install/1.7/Fri Jul 13 22:31:01 2007//
D/po////
/sitedoc.css/1.6/Sun Nov 11 16:32:59 2007//
/sitedoc.module/1.43/Thu Jan 17 21:56:40 2008//

Entries.Extra

/README.txt////*///
/sitedoc.info////*///
/sitedoc.install////*///
D/po///////
/sitedoc.css////*///
/sitedoc.module////*///

Entries.Extra.Old

/README.txt////*///
/sitedoc.info////*///
/sitedoc.install////*///
D/po///////
/sitedoc.css////*///
/sitedoc.module////*///

Entries.Old

/README.txt/1.1/Sat May 26 03:15:07 2007//
/sitedoc.info/1.5/Tue Jun 19 13:02:37 2007//
/sitedoc.install/1.7/Fri Jul 13 22:31:01 2007//
D/po////
/sitedoc.css/1.6/Sun Nov 11 16:32:59 2007//
/sitedoc.module/1.42/Wed Jan  2 00:37:51 2008//

Repository

contributions/modules/sitedoc

Root

:pserver:nancyw@cvs.drupal.org:/cvs/drupal-contrib

Template is empty

dww’s picture

Project: Update Status » CVS deploy
Version: 5.x-2.x-dev » 5.x-1.x-dev
Component: User interface » Code
Assigned: Unassigned » dww
Category: support » bug
Priority: Minor » Normal
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new945 bytes

Thanks for the info. The data you provided made me say to myself "in-con-CEIV-able". ;) So, I tried it locally, and lo, it's broken. Tee hee, whoops. ;)

Please try the attached patch to cvs_deploy DRUPAL-5. Works for me. Please confirm it's working for you, too. Thanks. Note, because of the differences between D5 and D6, this bug isn't present in the D6 version of cvs_deploy.

Also, note: there's a slightly weird edge case where if you view admin/build/modules or admin/logs/updates when you have no available update data, cvs_deploy can't resolve 'HEAD' into something better. So, you should reload either page as soon as you visit the admin/logs/updates page or "Check manually". Nothing much I can really do about that, unfortunately.

dww’s picture

Title: Update available? On my own module » Can't resolve HEAD releases to real versions
nancydru’s picture

Oh, my, I wasn't doing something wrong? Which is more "in-con-CEIV-able"? I didn't think I saw this in D6; I will look more closely though.

I'll give the patch a try shortly (have to get my mother up and fed first).

Thanks for checking into this.

nancydru’s picture

StatusFileSize
new129.06 KB

Here's a current screenshot. It seems to have the correct version, but still incorrectly indicates that there is an update available (and US too?).

dww’s picture

Status: Needs review » Fixed

That's a separate problem. cvs_deploy is trying to specify the 'datestamp' attribute for -dev releases based on the modification times on the CVS/Entries files. http://drupal.org/node/158526 u_s then has to compare these against the datestamps of the tarballs on d.o in the XML release history files. it's entirely possible that due to timezone or other clock skew, if your checkouts are from pretty close to the same time, that cvs_deploy thinks the d.o versions are newer. I don't really care, either: u_s is not meant to be perfect when people use -dev and branches. In fact, it's sort of against Earl and my better judgement that it supports them at all -- we'd rather maintainers created and users used, official releases from tags. 1 possible work around that should even work on windoze: remove a file from your workspace (e.g. the README.txt) and checkout again, and CVS/Entries will be newer than the latest tarball on d.o and u_s will be happy.

Given that my patch fixes the actual bug here, I committed to DRUPAL-5.

nancydru’s picture

Okay, go for it.

I do create, and prefer to use, official releases. However, there are times when I want to put several little fixes into a -dev before creating a new tag.

dww’s picture

Yes, of course. You should batch small changes into official releases. My point is, why should I spend any time worrying about what update_status tells you about your own -dev code?

nancydru’s picture

You shouldn't. But, on the other hand, I don't worry about US - it's in good hands.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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