ROOT tag file contains hard coded paths

[ferdymercury]

• Checked for duplicates

Describe the bug

The ROOT doxygen tag files posted in https://root.cern/reference/ contain full hard coded paths to /home/sftnight/... that are not available from outside users.

Expected behavior

No hard-coded paths are shown there.

To Reproduce

• Download the ROOT.tag from the link above
• grep sftnight ROOT.tag

Maybe playing with STRIP_FROM_PATH in the Doxyfile could solve this.

Additional context

doxygen/doxygen#9185

[couet]

@ferdymercury : Thanks for reporting. Do you have an idea what STRIP_FROM_PATH should be / you know that matter better than me. I am happy to merge a PR fixing it if you have one.

[ferdymercury]

We need to add two elements to STRIP_FROM_PATH, one is:
/home/sftnight/build/workspace/root-makedoc-master/rootspi/rdoc/master_TMP/

and the other:
/home/sftnight/build/workspace/root-makedoc-master/rootspi/rdoc/src/master/

Now, I do not know if this can be done in the Doxyfile in a clean way, by specifying the "current" directory or using the environment variable $(DOXYGEN_OUTPUT_DIRECTORY) or "../../../", not sure...

[couet]

in Doxyfile we have:

STRIP_FROM_PATH        = $(DOXYGEN_STRIP_PATH)

And DOXYGEN_STRIP_PATH is defined in Makefile the following way:

DOXYGEN_STRIP_PATH := $(shell cd $(PWD)/../..; pwd)

Is that ok ? looks fine to me. No ?

[ferdymercury]

Yes, but that's the ROOT trunk source directory.

Somehow, in ROOT.tag we are also including files from the rootspi repository, and that's maybe not in the STRIP_PATH right now? (Probably these are generated on the fly by the script or something like that?)

If not, maybe this is a bug in doxygen. We could report it if we find a minimal reproducer.

[couet]

I see it is /home/sftnight/build/workspace/root-makedoc-master/rootspi/rdoc/master_TMP/pyzdoc/ . So the pyzdoc which I guess should e also put in STRIP_FROM_PATH somehow ?

[couet]

Yes that's it because that folder is added in the list of folders to be scanned by doxygen at the end of "makeinput.sh`

[ferdymercury]

Yes. STRIPfrompath accepts an array of paths (space separated), not just a single path.

[ferdymercury]

I filed a bug report in doxygen doxygen/doxygen#9221

[couet]

have you tried STRIP_FROM_PATH ? Why do you think there is a bug ?

[ferdymercury]

yes, I tried and the path is still there