[Bug 1549] It must be easier to write tests

bugzilla-daemon at rtems.org bugzilla-daemon at rtems.org
Fri Jun 11 08:31:52 UTC 2010


https://www.rtems.org/bugzilla/show_bug.cgi?id=1549





--- Comment #6 from Sebastian Huber <sebastian.huber at embedded-brains.de>  2010-06-11 03:31:45 ---
> --- Comment #5 from Ralf Corsepius <ralf.corsepius at rtems.org>  2010-06-11 03:10:32 ---
> Sebastian, vertainly, there is much of room for improvment, but this PR to me
> is close to qualify as "clueless" ranting.
> 
> I am considering this PR to be not helpful, I am seriously considering to close
> it as "Invalid" and/or "SPAM".

You may be right, but silence will improve nothing.

> Some comments:
> 
>> Grouping by directory or file name prefix is not that different.
> Wrong, it is substantially different.

Ok, it depends on the scope.  For humans its not that different.

>> AM_CPPFLAGS += -I$(top_srcdir)/../support/include
>>
>> # This is a hack.  It makes it hard (impossible?) to add multiple programs to
>> # one Makefile.am. 
> Wrong, you don't seem to understand.
> 
> Correct, it is impossible to add multiple programs to one Makefile.am, but the
> cause is not AM_CPPFLAGS, the cause is make-exe in "*.cfg" and the MANAGERS.

It seem that it was not clear enough were this comment belongs to.  Maybe this
is better:

# A hack follows (goes up to the next comment).  It makes it hard (impossible?)
# to add multiple programs to one Makefile.am.  I guess the reason for this is
# the creation of *.ralf files via $(bsp-post-link).  There has to be another
# way to create *.ralf files from *.exe files.  It would be better to have a
# shell script for this and not a make function.  The *.ralf files created this
# way are also not that useful since they won't be installed.  Only the *.exe
# files will be installed.

LINK_OBJS = $(sp01_OBJECTS) $(sp01_LDADD)
LINK_LIBS = $(sp01_LDLIBS)

sp01$(EXEEXT): $(sp01_OBJECTS) $(sp01_DEPENDENCIES)
    @rm -f sp01$(EXEEXT)
    $(make-exe)

# Hack end.

>> Yes, we have the texinfo files.  Compare the update frequency of the source
>> code path with the documentation tree.  It is a constant burden to keep this
>> synchronized.  Only few people touch the texinfo files at all.  We should move
>> to source code embedded documentation (e.g. Doxygen) entirely.  You can write
>> book style documentation with Doxygen easily.  See
> Apples and oranges:
> * RTEMS doc/ and the doxygen comments serve different purposes.

They are only two means to reach a goal.

> * texinfo is just _one_ output format of documentation.

We should really think about this.

> 
> That said, I agree insofar, as that source-code documentation should be removed
> from doc/ and be replaced with doxygen generated documentation in *info* 
> format (!)

Doxygen can generate Latex, HTML and man pages.  I am not aware that it can
generate info format.

> 
> Finally: IMO, the current doxygen stuff (esp. the online incarnation) is simply
> plain broken and invalid.
> 

To say that something is broken and invalid you need something that specifies
the good and valid.

-- 
Configure bugmail: https://www.rtems.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.



More information about the bugs mailing list