Bug 319101

Summary:	XScreenSaver.ad uses latin-1 encoding (XScreenSaver.ad.tail too)
Product:	[Fedora] Fedora	Reporter:	Doncho Gunchev <dgunchev>
Component:	xscreensaver	Assignee:	Mamoru TASAKA <mtasaka>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	low	Docs Contact:
Priority:	low
Version:	rawhide	Keywords:	Reopened
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	5.03-10.fc7	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2007-10-08 15:02:39 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Doncho Gunchev 2007-10-04 19:16:28 UTC

Description of problem:
XScreenSaver.ad and XScreenSaver.ad.tail use latin-1 encoding - look 
for 'Copyright . 1991-2006 by', where the dot is latin1 Â© /(c), the copyright 
sign, in case bugzilla encodings get wrong again/.

Version-Release number of selected component (if applicable):
xscreensaver-base-5.03-9.fc8, I'm filling against xscreensaver because there's 
no xscreensaver-base.

How reproducible:
always

Steps to Reproduce:
1. install xscreensaver-base-5.03-9.fc8
2. cd /etc/xscreensaver
3. grep -R Copyright .

Actual results:
one needs iconv:
grep -R Copyright . | iconv -f latin1

./XScreenSaver.ad.tail:*splash.body.label:              Copyright Â© 1991-2006 
by
./XScreenSaver.ad:*splash.body.label:           Copyright Â© 1991-2006 by

to see this correctly

Expected results:
should not need iconv

Additional info:
Same is true for /usr/bin/xscreensaver-text, xscreensaver-getimage-video and 
xscreensaver-getimage-file, but there's some code that looks using that 
encoding plus some hacks and I can't tell what would it take to convert them 
to UTF-8.
All man pages look OK, Â© is in UTF-8.

Comment 1 Mamoru TASAKA 2007-10-05 04:30:19 UTC

Currently marking as NOTABUG.

* XScreenSaver.ad is really a configuration file, and 
  *splash.body.label string is used in xscreensaver-demo splash
   dialog.

Comment 2 Doncho Gunchev 2007-10-05 06:05:30 UTC

Excuse my curiosity, but what does "currently marking as" mean? Can bugzilla 
reopen bugs by itself? I don't really care if this never gets fixed, I'm just 
curious :-)

Comment 3 Mamoru TASAKA 2007-10-05 06:15:41 UTC

That is:

Current XScreenSaver.ad works, and I am not sure this file
can be converted into UTF-8 with no problem (this is not just
a document). It may require some investigation, so I wrote
"for now" I close as NOTABUG.

Comment 4 Mamoru TASAKA 2007-10-06 14:47:52 UTC

Will be fixed in 5.03-10.fc{7,8} (with other bugs)

Comment 5 Fedora Update System 2007-10-08 15:02:37 UTC

xscreensaver-5.03-10.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.