Bug 6986 - Frequent CVS conflicts in doc/spec/hal-spec.html
Summary: Frequent CVS conflicts in doc/spec/hal-spec.html
Status: RESOLVED FIXED
Alias: None
Product: hal
Classification: Unclassified
Component: spec (show other bugs)
Version: unspecified
Hardware: x86 (IA32) Linux (All)
: high normal
Assignee: Danny Kukawka
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-05-21 02:18 UTC by Torsten Schönfeld
Modified: 2006-07-24 05:04 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Conflicts in hal-spec.html (15.22 KB, patch)
2006-06-03 07:56 UTC, Torsten Schönfeld
Details | Splinter Review
patch to fix build hal-spec.html (3.11 KB, patch)
2006-07-21 07:32 UTC, Danny Kukawka
Details | Splinter Review

Description Torsten Schönfeld 2006-05-21 02:18:42 UTC
When I update hal from CVS, I very often get conflicts in
doc/spec/hal-spec.html.  If this is a generated file, please don't put it into
the repository.
Comment 1 Danny Kukawka 2006-05-24 01:26:51 UTC
This is a from xml generated file and we have this file in CVS to have always a 
up-to-date version available via a link from the project website (http://cvs.
freedesktop.org/*checkout*/hal/hal/doc/spec/hal-spec.html)

What kind of conflicts do you get?
Comment 2 Torsten Schönfeld 2006-05-24 04:37:20 UTC
(In reply to comment #1)
> This is a from xml generated file and we have this file in CVS to have always a 
> up-to-date version available via a link from the project website (http://cvs.
> freedesktop.org/*checkout*/hal/hal/doc/spec/hal-spec.html)

I see.

> What kind of conflicts do you get?

I don't remember.  As soon as one occurs again, I'll let you know the details.
Comment 3 Torsten Schönfeld 2006-06-03 07:56:42 UTC
Created attachment 5793 [details] [review]
Conflicts in hal-spec.html

Here we go.  Conflicts I just got from updating to HEAD.
Comment 4 Danny Kukawka 2006-07-21 07:32:01 UTC
Created attachment 6293 [details] [review]
patch to fix build hal-spec.html

I already saw the problem (at least in Opera/Firefox/Epiphany and in the file
while update the CVS) with invalid chars in the html file. IMO there is a
problem with xmlto and build the hal-spec.html.

This patch change the build process back to docbook2html and add xmllint to
resolve the usage of xinclude. This should avoid the invalid chars.
Comment 5 Danny Kukawka 2006-07-24 05:04:47 UTC
commited to CVS HEAD:

2006-07-24  Danny Kukawka  <danny.kukawka@web.de>

        Fixed build hal-spec.html to avoid invalid chars in the html page and
        closed fd.o bug #6986:

        * configure.in: replaced check for xmlto with old check for
        docbook2html and additional check for xmllint

        * doc/spec/Makefile.am: added xmllint to resolve xinclude for docbook,
        readded docbook2html command



Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.