8547 – kde example not bash compliant (right?)

Bug 8547 - kde example not bash compliant (right?)

Summary: kde example not bash compliant (right?)

Status:	RESOLVED FIXED

Alias:	None

Product:	[OLD] Docs-developer
Classification:	Unclassified
Component:	Eclass HOWTO (show other bugs)
Hardware:	All All

Importance:	High normal (vote)
Assignee:	Docs Team

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2002-09-29 17:08 UTC by Denys Duchier
Modified:	2002-11-12 18:40 UTC (History)
CC List:	1 user (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Denys Duchier 2002-09-29 17:08:51 UTC

code listing 14 (among others) begins with:   
   
kde_src_compiler() {   
    section myconf:   
...   
   
(1) it's not really what's in kde.eclass (2) that's not correct bash   
syntax, is it?  sorry to be a nuisance :-(

Comment 1 SpanKY gentoo-dev

2002-09-30 00:09:26 UTC

eclass's

Comment 2 John Davis (zhen) (RETIRED) gentoo-dev

2002-10-02 18:21:34 UTC

What does this have to do w/ documentation?

//ZhEN

Comment 3 Denys Duchier 2002-10-03 01:19:43 UTC

the code listing "documents" an eclass. 
this "documentation" is wrong.  It suggests invalid syntax to the reader.

Comment 4 John Davis (zhen) (RETIRED) gentoo-dev

2002-10-24 14:32:38 UTC

What would be the recommended fix for this?

//ZhEN

Comment 5 SpanKY gentoo-dev

2002-10-24 19:52:15 UTC

either update the code listing to look like that of the current kde.eclass, or 
change 'section myconf:' to 'local myconf;' and remove 
'--with-qt-dir=${QTDIR}"' in the third line ...

Comment 6 John Davis (zhen) (RETIRED) gentoo-dev

2002-11-12 18:40:49 UTC

fixed in cvs..
For the record - the formatting of this guide is AWFUL. Come on people..

//ZhEN