Community
Participate
Working Groups
Created attachment 183959 [details] Changelog plugin PSF Steps to reproduce: 1. Checkout ChangeLog code (may use attached PSF) 2. Remove fragment.xml from org.eclipse.linuxtools.changelog.cparser 3. Add new class in src/org/eclipse/linuxtools/changelog/parsers (Test.java say) 4. Select project org.eclipse.linuxtools.changelog.cparser 5. "Project" => "Prepare Changelog" Result: 2010-11-26 Severin Gehwolf <sgehwolf@redhat.com> * fragment.xml: Removed. * src/org/eclipse/linuxtools/changelog/parsers/Test.java: Expected: 2010-11-26 Severin Gehwolf <sgehwolf@redhat.com> * fragment.xml: Removed. * src/org/eclipse/linuxtools/changelog/parsers/Test.java: New file.
$ rpm -q eclipse-changelog eclipse-changelog-2.6.7-5.fc14.x86_64 .changelog.core 2.6.7 .changelog.parsers.java 2.6.3
There is a bug in the GNU Formatter for ChangeLog whereby the last line in the entry does not add any default content. The entry is sorted so removed files come first, then new files, then modified files. If there are no modified files and there are new files or removed files, the last entry will be incorrect because the default content is the "New file" or "Removed file" message. This has been corrected in trunk and should be in branch 0.7 when it is cut.
(In reply to comment #2) > There is a bug in the GNU Formatter for ChangeLog whereby the last line in the > entry does not add any default content. The entry is sorted so removed files > come first, then new files, then modified files. If there are no modified > files and there are new files or removed files, the last entry will be > incorrect because the default content is the "New file" or "Removed file" > message. > > This has been corrected in trunk and should be in branch 0.7 when it is cut. Thanks Jeff. I'll add a regression test to the GNU Formatter tests for those cases.
(In reply to comment #3) > Thanks Jeff. I'll add a regression test to the GNU Formatter tests for those > cases. Regression test for this bug is part of the GNUFormat test suite in trunk (Revision 27047).