Bug 1909 - Official update request: Provide a tainted build of freetype2
Summary: Official update request: Provide a tainted build of freetype2
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 1
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 1549
  Show dependency treegraph
 
Reported: 2011-06-24 10:39 CEST by Ahmad Samir
Modified: 2014-05-08 18:05 CEST (History)
2 users (show)

See Also:
Source RPM: freetype2
CVE:
Status comment:


Attachments

Description Ahmad Samir 2011-06-24 10:39:42 CEST
Mageia 1 was shipped without a tainted build of freetype2 (mainly due to lack of time before release). This should provide a tainted build; for the packages in core, this is just a rebuild, i.e. nothing is changed with respect to the package in core.

Test case:
- Install the freeteype2 package(s) from the core/updates_testing
- Log out -> log in, nothing should change with respect to how fonts look

For tainted, this enables a feature that didn't exist before, so there can't be any regressions.
Ahmad Samir 2011-06-24 10:40:16 CEST

Assignee: bugsquad => qa-bugs

Ahmad Samir 2011-06-24 10:41:21 CEST

Blocks: (none) => 1549

Comment 1 Dave Hodgins 2011-06-24 19:39:23 CEST
I've installed libfreetype6-2.4.4-5.1.mga1.tainted.i586 and
freetype2-demos-2.4.4-5.1.mga1.tainted.i586.

Logged out/in.  No obvious difference, on my system.

Presumably there is a way to activate the subpixel hinting that I'm missing,
otherwise what does it accomplish?

CC: (none) => davidwhodgins

Comment 2 Ahmad Samir 2011-06-24 21:24:40 CEST
A freetype2 build with subpixel rendering enabled should enable you to use the fontconfig lcdfilter settings (which would be ignored altogether with a non-subpixel-enabled freetype2 build).

I forgot to say, this only affects GTK+ apps, Qt4/KDE apps aren't affected at all.
Comment 3 Dave Hodgins 2011-06-25 01:40:38 CEST
I installed task-gnome, and used System/Preferences/Appearance to turn on
subpixel smoothing for the fonts.  After logging out/back in, I don't see
any noticeable difference on my crt monitor.

How can someone check to see if subpixel rendering is being used?
Comment 4 Ahmad Samir 2011-06-25 04:12:39 CEST
IIUC, the subpixel rendering is more for LCD monitors, though they say "some users reported is made fonts looks better or CRT monitors too".
Comment 5 Dave Hodgins 2011-06-25 08:59:56 CEST
If there is anyone with a 64 bit system available to confirm it installs
cleanly there, I think that would be enough to allow pushing from
tainted-updates-testing to tainted-updates.
Comment 6 Dave Hodgins 2011-06-25 09:15:11 CEST
Sorry, should have included the packages that needs to be tested on x64/pushed
from tainted updates testing to tainted updates are
freetype2-demos
libfreetype6
libfreetype6-devel
libfreetype6-static-devel.

I think simply installing them, and confirming the display on a gnome system
doesn't get messed up, should be enough.
Comment 7 Ahmad Samir 2011-06-25 14:47:08 CEST
The whole point of providing a tainted build of freetype2 is offering an "opt-in" option for users who actually used subpixel-rendering-enabled freetype2 from PLF.

I don't see how the display on a GNOME installation would get messed up by a freeype2 tainted build.
Comment 8 Dave Hodgins 2011-06-27 04:02:31 CEST
I didn't mean to imply it would mess things up.  Just that the installation
should also be tested on a 64 bit system.  Once that's done, the packages
listed in comment 6 should be pushed to tainted updates.
Comment 9 Hrvoje Senjan 2011-06-27 07:31:50 CEST
I did test packages from Tainted Updates testing , and they are working :)

CC: (none) => hrvoje.senjan

Comment 10 Dave Hodgins 2011-06-27 22:47:20 CEST
Then the following packages are ready to push from Tainted Updates Testing
to Tainted updates on both i586 and x86-64.

freetype2-demos
libfreetype6
libfreetype6-devel
libfreetype6-static-devel
Comment 11 Nicolas Vigier 2011-06-30 14:16:33 CEST
Do you have an advisory text for this update ?

CC: (none) => boklm

Comment 12 Ahmad Samir 2011-07-02 02:09:33 CEST
Proposed Advisory text:
=========================================
FreeType2 as shipped in Mageia 1 doesn't have subpixel rendering enabled (a build time option). This update provides FreeType2 packages with sub-pixel rendering enabled only in the tainted section as this feature is covered by some software patents which violate copyright laws in some countries in the world.
=========================================
Comment 13 Nicolas Vigier 2011-07-05 00:16:46 CEST
I don't think it violates "copyright laws", but maybe "patent laws".

Maybe we can just say "feature is covered by some software patents"  ?
Comment 14 Nicolas Vigier 2011-07-05 00:19:53 CEST
pushed to updates.

Status: NEW => RESOLVED
Resolution: (none) => FIXED

Comment 15 Ahmad Samir 2011-07-05 00:45:37 CEST
(In reply to comment #13)
> I don't think it violates "copyright laws", but maybe "patent laws".
> 
> Maybe we can just say "feature is covered by some software patents"  ?

Yes, sorry, I was probably too sleepy.
Comment 16 Dave Hodgins 2011-07-05 04:01:51 CEST
I've noticed that the packages
freetype2-demos
libfreetype6-devel
libfreetype6-static-devel
libfreetype6
are in Core Updates Testing.  Does the build system now generate both the
Core and tainted versions at the same time?

If so should these packages be pushed too, along with the srpm?
Comment 17 Ahmad Samir 2011-07-05 04:19:52 CEST
(In reply to comment #16)
> I've noticed that the packages
> freetype2-demos
> libfreetype6-devel
> libfreetype6-static-devel
> libfreetype6
> are in Core Updates Testing.  Does the build system now generate both the
> Core and tainted versions at the same time?
> 

No, it doesn't; we submit in both locations every time.

@sysadm: please move these ones to core/updates too.

> If so should these packages be pushed too, along with the srpm?
Nicolas Vigier 2014-05-08 18:05:49 CEST

CC: boklm => (none)


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