Sounds like a great module. Because of that, I've cleaned-up the coding-style. No functional changes.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | auto_username-DRUPAL-5.patch | 9.43 KB | sun |
| #2 | auto_username-HEAD.patch | 2.99 KB | sun |
| auto_username-DRUPAL-5.patch | 9.29 KB | sun |
Comments
Comment #1
Crell commentedThere were a few failed hunks, so I think you were working against the very initial release. I've done some bug fixing since then. I committed the parts that applied. Thanks.
Comment #2
sunRe-rolled against current HEAD / DRUPAL-5.
Comment #3
Crell commentedThe D5 patch fails on every hunk. :-) Probably you're not working on the latest CVS. Also, don't bother with HEAD. I virtually never use it. Drupal 5 development will happen in the DRUPAL-5 branch.
Comment #4
sunNow I'm a bit confused, since I did a fresh CVS checkout of auto_username in front of creating the last patch against DRUPAL-5... are you sure?
Comment #5
sunI can't help - last patch applies cleanly on a fresh CVS checkout of DRUPAL-5... maybe you've set a sticky revision/date?
Comment #6
Crell commentedbah. The problem was my working copy was using HEAD instead of DRUPAL-5. Silly CVS. Should all be fixed and synced up now. Thanks.
Comment #7
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.