Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 136109 - ck-sources-2.6.17_p1 [Version Bump 2]
Summary: ck-sources-2.6.17_p1 [Version Bump 2]
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: x86-kernel@gentoo.org (DEPRECATED)
URL: http://members.optusnet.com.au/ckoliv...
Whiteboard:
Keywords: EBUILD
: 137310 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-06-08 12:53 UTC by Peter Hyman
Modified: 2006-06-22 10:01 UTC (History)
8 users (show)

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


Attachments
ck-sources-2.6.16_p12.ebuild (ck-sources-2.6.16_p12.ebuild,1.40 KB, text/plain)
2006-06-08 12:54 UTC, Peter Hyman
Details
ck-sources-2.6.16_p12.ebuild (ck-sources-2.6.16_p12.ebuild,1.44 KB, text/plain)
2006-06-10 04:24 UTC, Peter Hyman
Details
ck-sources-2.6.16_p12.ebuild (ck-sources-2.6.16_p12.ebuild,1.50 KB, text/plain)
2006-06-10 05:36 UTC, Peter Hyman
Details
ck-sources-2.6.16_p12.ebuild (ck-sources-2.6.16_p12.ebuild,1.50 KB, text/plain)
2006-06-10 05:51 UTC, Peter Hyman
Details
ck-sources-2.6.16_p12.ebuild (ck-sources-2.6.16_p12.ebuild,1.50 KB, text/plain)
2006-06-11 17:26 UTC, Peter Hyman
Details
ck-sources-2.6.17_p1.ebuild (ck-sources-2.6.17_p1.ebuild,1.49 KB, text/plain)
2006-06-19 11:11 UTC, Peter Hyman
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Hyman 2006-06-08 12:53:53 UTC
includes latest patchset, but some important changes to ebuild.
1) have to exclude patches 2400, 25*, 26*, and 27*, since the ck patchset won't apply.
2) removed redundant call to detect-version.
Comment 1 Peter Hyman 2006-06-08 12:54:55 UTC
Created attachment 88712 [details]
ck-sources-2.6.16_p12.ebuild
Comment 2 Peter Hyman 2006-06-10 04:23:00 UTC
With revisions posted in bug #136076, all but three genpatches can be applied. 2500, 2505, and 2605 fail. The revised ebuild required changes to kernel-2.eclass to function as expected. Otherwise, ck patch will not apply at all. New patch introduces the use of UNIPATCH_GENPATCHES_LAST flag and alters the exclude list to specifically mask the patches that won't apply.
Comment 3 Peter Hyman 2006-06-10 04:24:19 UTC
Created attachment 88830 [details]
ck-sources-2.6.16_p12.ebuild

this ebuild will not work with current kernel-2.eclass. It is intended for the kernel-2.eclass I patched only.
Comment 4 Peter Hyman 2006-06-10 05:36:44 UTC
Created attachment 88837 [details]
ck-sources-2.6.16_p12.ebuild

remove K_PREPATCHED and UNIPATCH_STRICTORDER to allow ck-patchset to get applied first followed by gentoo patches. Thanks, dsd, for helping me resolve this. With revised order, some genpatches won't apply, 2500, 2505, and 2605. As of now, no need for new variable to affect patch order. For some reason, the two now-removed variables were held over. They are not needed.
Comment 5 Peter Hyman 2006-06-10 05:51:31 UTC
Created attachment 88839 [details]
ck-sources-2.6.16_p12.ebuild

mistakenly had 2505 in exclude list, but that patch does not exist. my bad.
Comment 6 Peter Hyman 2006-06-11 07:59:59 UTC
fwiw, it's possible to include the extras patchset as well with the following exclusions: 4200 4205 (fbsplash and vesa-tng), and 4405 (alpha-sysctl-uac.patch). The fbsplash patch requires -F3 in order to patch, and it works. The beyond-sources contain the fbsplash and vesa-tng patches. See bug #103354.

Maintainer may wish to consider testing and using.
K_WANT_GENPATCHES="base extras"
UNIPATCH_EXCLUDE="${UNIPATCH_EXCLUDE} 10 2500 2605 4200 4205 4405"

are the lines that need changing.

A consideration would be to create a separate local use flag to control whetner either or both of the genpatches get applied. The result of this would be a hybrid ck and gentoo kernel. Pretty neat.
Comment 7 Peter Hyman 2006-06-11 08:46:56 UTC
MY_P is not used. Can remove. Or, if you need to keep it, have it contain the ck## revision so it can inserted into EXTRAVERSION.

# A few hacks to set ck version via _p instead of -r
MY_P=${PV/*_p/}  # replace this into EXTRAVERSION below
MY_PR=${PR/r/-r}
MY_PR=${MY_PR/-r0/}
EXTRAVERSION=-ck${MY_P}${MY_PR}
KV_FULL=${OKV}${EXTRAVERSION}
KV_CK=${KV_FULL/-r*/}
Comment 8 Peter Hyman 2006-06-11 17:26:59 UTC
Created attachment 88945 [details]
ck-sources-2.6.16_p12.ebuild

Cleanups: put description and homepage at top, remove unnecessary MY_P variable.
Enhancement: Added patches from gentoo extra and proper excludes.
Comment 9 Andrew Gaydenko 2006-06-15 05:01:19 UTC
Peter,

Is it safe to try your ebuild for ordinary (like me) user?
Comment 10 Peter Hyman 2006-06-15 07:06:19 UTC
(In reply to comment #9)
> Peter,
> 
> Is it safe to try your ebuild for ordinary (like me) user?
> 

I have compiled and run the ck12 sources and have noticed no problems. Obviously, keep your current working kernel in your lilo/grub configuration in case you get a panic. But if you have been using ck11, this does not have big differences.
Comment 11 Andrew Gaydenko 2006-06-15 10:56:09 UTC
(In reply to comment #10)
> I have compiled and run the ck12 sources and have noticed no problems.
> Obviously, keep your current working kernel in your lilo/grub configuration in
> case you get a panic. But if you have been using ck11, this does not have big
> differences.
> 

Thanks, I'm under p12 now (~x86).
Comment 12 Jacob Braun 2006-06-19 07:59:50 UTC
2.6.17-ck1 is out now.
Comment 13 Peter Hyman 2006-06-19 11:09:01 UTC
I tried hacking in the fbsplash and vesatng patches, but they absolutely are not working with this version -- even with iphitus' hack. So, the new version will still not include the framebuffer splash patches. Maybe spock (hint, hint) will get a chance to look at it. The beyond-sources4.1 are based on ck11 set, and they contain the fbsplash.
Comment 14 Peter Hyman 2006-06-19 11:11:29 UTC
Created attachment 89547 [details]
ck-sources-2.6.17_p1.ebuild

please consider these unofficial as they have not been reviewed by the kernel team. Especially wrt the application and exclusion of genpatches.
Comment 15 charlielaw 2006-06-19 15:33:24 UTC
*** Bug 137310 has been marked as a duplicate of this bug. ***
Comment 16 Micheal Marineau (RETIRED) gentoo-dev 2006-06-22 09:08:28 UTC
Sorry I have been a bit MIA for the last week or two, I'll get this bumped in portage shortly.

As for the interest in putting fbsplash and vesa-tng into ck-sources it's not going to happen, sorry. There was a developer maintaining an out of portage ebuild for that but I think he stopped a while back. You are perfectly free to patch it in yourself of course but will require resolving a few patch failures usually.
Comment 17 Micheal Marineau (RETIRED) gentoo-dev 2006-06-22 10:01:14 UTC
/me actually reads the bug correctly this time around.

Peter,
I missed that you were suggesting having all of extras except for fbsplash and vesa-tng added in. I'm going to continue using only 'base' as this is simply supposed to be the ck patchset and base is only there to make it easy to pull in security and serious bug fixes.

Thanks for cleaning up the ebuild. STRICTORDER and PREPATCHED were left over from before I started maintaining the ebuild and I never bothered to look at what they were doing :-P

2.6.16-ck12 and 2.6.17-ck1 are now in CVS.