Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Often requested, simple to implement: An easy and quick way to track a project in RSS. Patch adds that.
After the patch the line goes like
» View CVS messages (RSS).
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | drupal.addCVS_RSSLink.patch_1.txt | 673 bytes | halkeye |
| #2 | addCVS_RSSlink.patch_0.txt | 640 bytes | Bèr Kessels |
| addCVS_RSSlink.patch.txt | 641 bytes | Bèr Kessels |
Comments
Comment #1
halkeye commentedDidn't work for me
your $query has a "?" in it., taking out the ?, everything works fine.
Comment #2
Bèr Kessels commentedremoved the ?
Comment #3
halkeye commentedseriously man, you need to actually test this
replaced the "." with "," before the query.
Comment #4
dwwlatest patch seems reasonable upon simple inspection. however, i'm currently on vacation until july 16th, with no good means to actually test anything like this. i refuse to commit changes to the project module that i can't test, so this will either have to wait, or nedjo/dries will have to test and commit it.
furthermore, there's active effort towards making the project module more RCS-independent (http://drupal.org/node/70125) and this particular patch is a step away from that goal. this is just a minor addition to an existing problem, so i'm not going to refuse it on those grounds. however, in general, i'd prefer if new patches that touch CVS stuff do it in a way that moves CVS-specific functionality out of project and into cvslog, where it belongs. for example, adding this link could be done via hook_nodeapi() in cvs.module...
thanks,
-derek
Comment #5
aclight commentedThis is a duplicate of #31994: Add a link to CVS RSS feeds from project node page.