Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 380871 - media-video/vlc-9999 - Failed Patch: 010_all_freetype-font.patch !
Summary: media-video/vlc-9999 - Failed Patch: 010_all_freetype-font.patch !
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: Normal normal (vote)
Assignee: Alexis Ballier
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-27 18:35 UTC by cmartin
Modified: 2011-09-08 18:42 UTC (History)
2 users (show)

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


Attachments
/var/tmp/portage/media-video/vlc-9999/temp/010_all_freetype-font.patch.out (010_all_freetype-font.patch.out,4.39 KB, text/plain)
2011-08-27 18:39 UTC, cmartin
Details
Complete build log (media-video:vlc-9999:20110827-182849.log,5.72 KB, text/plain)
2011-08-27 18:41 UTC, cmartin
Details
ebuild environment file (environment,134.88 KB, text/plain)
2011-08-27 18:42 UTC, cmartin
Details
010_all_freetype-font.patch for latest VLC trunk (010_all_freetype-font.patch,753 bytes, patch)
2011-08-28 15:45 UTC, David Korth
Details | Diff
300_all_pic.patch for latest VLC trunk (300_all_pic.patch,5.85 KB, patch)
2011-08-28 15:50 UTC, David Korth
Details | Diff
320_all_cachegen.patch for latest VLC trunk (320_all_cachegen.patch,1.10 KB, patch)
2011-08-28 15:51 UTC, David Korth
Details | Diff
300_all_pic.patch for latest VLC trunk, take 2 (300_all_pic.take2.patch,5.76 KB, text/plain)
2011-08-28 16:08 UTC, David Korth
Details
300_all_pic.patch for latest VLC trunk, take 3 (300_all_pic.take3.patch,5.83 KB, patch)
2011-08-28 16:11 UTC, David Korth
Details | Diff
320_all_cachegen.patch for latest VLC trunk, take 2 (320_all_cachegen.take2.patch,980 bytes, patch)
2011-08-28 16:15 UTC, David Korth
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description cmartin 2011-08-27 18:35:06 UTC
The freetype-font.patch can't find the file vlc/modules/misc/text_renderer/freetype.c to patch it

Reproducible: Always

Steps to Reproduce:
1.Add the line "media-video/vlc **" to /etc/portage/package.keywords
2.emerge media-video/vlc
3.Compilation fails in patching
Actual Results:  
Compilation fails

Expected Results:  
The patch should succeed
Comment 1 cmartin 2011-08-27 18:39:21 UTC
Created attachment 284819 [details]
/var/tmp/portage/media-video/vlc-9999/temp/010_all_freetype-font.patch.out

Output from the patch attempt
Comment 2 cmartin 2011-08-27 18:41:26 UTC
Created attachment 284821 [details]
Complete build log
Comment 3 cmartin 2011-08-27 18:42:38 UTC
Created attachment 284823 [details]
ebuild environment file
Comment 4 David Korth 2011-08-28 15:45:12 UTC
Created attachment 284891 [details, diff]
010_all_freetype-font.patch for latest VLC trunk

The text_renderer module was moved from modules/misc/ to modules/ in VLC commit 0d648eeea4101aa92f7b02d6ad114c6d086c9f82 on August 4.

This updated patch changes the paths so the text renderer module is patched correctly.

Patches 300_all_pic.patch and 320_all_cachegen.patch no longer apply either, so I'll fix those next.
Comment 5 David Korth 2011-08-28 15:50:43 UTC
Created attachment 284893 [details, diff]
300_all_pic.patch for latest VLC trunk
Comment 6 David Korth 2011-08-28 15:51:16 UTC
Created attachment 284895 [details, diff]
320_all_cachegen.patch for latest VLC trunk
Comment 7 David Korth 2011-08-28 16:08:15 UTC
Created attachment 284897 [details]
300_all_pic.patch for latest VLC trunk, take 2

Spacing errors prevented the patch from applying correctly.
Comment 8 David Korth 2011-08-28 16:11:38 UTC
Created attachment 284899 [details, diff]
300_all_pic.patch for latest VLC trunk, take 3

Another minor fix for 300_all_pic.patch.
Comment 9 David Korth 2011-08-28 16:15:15 UTC
Created attachment 284901 [details, diff]
320_all_cachegen.patch for latest VLC trunk, take 2

Same as before, but with spacing fixes.
Comment 10 Alexis Ballier gentoo-dev 2011-09-08 18:42:48 UTC
i updated the live ebuild for recent changes... hadn't seen that bug :( thanks anyway