New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generated Doxyfile differs from result of doxygen -u (Origin: bugzilla #702265) #5213

Open
doxygen opened this Issue Jul 2, 2018 · 2 comments

Comments

Projects
None yet
2 participants
@doxygen
Owner

doxygen commented Jul 2, 2018

status NEW severity minor in component doxywizard for ---
Reported in version 1.8.5-GIT on platform Other
Assigned to: Dimitri van Heesch

Original attachment names and IDs:

On 2013-06-14 13:49:42 +0000, Roland Clobus wrote:

The Doxyfile that is generated by doxywizard differs from the Doxyfile that is generated by doxygen.

Steps to reproduce:
1) Generate the files
$ doxygen -g sampleconfig
$ cp samplefonfig generated
$ doxygen -u sampleconfig
$ cp sampleconfig updated
$ doxywizard sampleconfig (and Save)
$ cp sampleconfig wizard
$ doxygen -u sampleconfig
$ cp sampleconfig wizardupdated
$ doxywizard (Save as wizardgenerated)

2) Look at the differences
$ diff generated updated
-> No difference
$ diff generated wizard
-> Many differences, mainly whitespace
$ diff generated wizardupdated
-> No difference
$ diff wizard wizardgenerated
-> A few differences in the settings of some parameters

Expected behaviour:
- No difference in the configuration files

I've used the git SHA: 8cd4425a523965a5541d46a84828fb7e16f98bae, but older versions (1.8.1.2-2 in Debian) also have this behaviour.

On 2013-06-14 17:10:13 +0000, albert wrote:

Work is in progress regarding the Doxyfile as generated by doxygen and the Doxywizard. The given command will be taken into account to to evaluate the results.

On 2013-08-24 16:08:08 +0000, albert wrote:

According to my knowledge this problem should be fixed now.

On 2013-08-24 16:50:32 +0000, Dimitri van Heesch wrote:

Setting to fixed as suggested.

On 2013-09-25 10:35:30 +0000, Roland Clobus wrote:

Thank you for the changes.

I did some tests with the current git version (SHA: bf8ecf7749d846a1f1b28ab01953ad55989b87a8)

Things have changed, but I still see differences.

$ diff generated wizard
-> Many differences, but they are only whitespace
$ diff wizard wizardgenerated
-> Different settings for the parameters: ABBREVIATE_BRIEF, CASE_SENSE_NAMES, FILE_PATTERNS, EXAMPLE_PATTERNS

On 2013-09-25 10:36:53 +0000, Roland Clobus wrote:

(Updated the version found from 1.8.4-GIT to 1.8.5-GIT)

On 2013-09-25 11:56:26 +0000, Roland Clobus wrote:

Created attachment 255679
Draft patch for making the whitespace identical

This patch contains 2 changes:
* convertToComment: the extra replace will make the empty comment lines contain just a hash, and not a hash and a space
* saveTopic: Not functional. It requires a new pure virtual function in the class Input : isNotEmpty. When a valid is empty, the writeValue method does not need to be called.

This patch can solve the whitespace differences that I see with
$ diff generated wizard

@doxygen doxygen added the doxywizard label Jul 19, 2018

@albert-github

This comment has been minimized.

Collaborator

albert-github commented Oct 4, 2018

I've just pushed a proposed patch, pull request #6538

See to it that line endings are correct and no extra white space (conform suggestions of Roland Clobus)

@albert-github

This comment has been minimized.

Collaborator

albert-github commented Oct 6, 2018

Pull request has been integrated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment