From Bugzilla Helper: User-Agent: Mozilla/5.0 Galeon/1.2.6 (X11; Linux i686; U;) Gecko/20020830 Description of problem: Currently, package/group/exclude lists are overwritten at the end of readKickstart. If %package lives in a %include'd file, all package info from the included file gets lost when the main call to readKickstart exits the main loop. The patch changes the behavior so that readKickstart appends package/group/exclude bits instead of assigning the current call's lists to the class attributes. This way no data gets lost, at least for packages. Version-Release number of selected component (if applicable): How reproducible: Always Steps to Reproduce: 1. put entire %package directive in a file 2. include that file from a ks.cfg using %include 3. Actual Results: the %package section's contents were overwritten with the empty package, group, exclude lists from the top level call. Expected Results: info from %package directives in *both* files should be preserved Additional info: there are obviously issues if users supply 2 or more conflicting sets of package selection info, but that's unavoidable IMHO. ordering of processing is predictable, so they'll just have to pay attention.
Created attachment 89192 [details] patch to change the behavior of the KickstartBase class as described in the original bug report
Thanks for the patch!
Applied
I'm going through Bugzilla closing some bugs that have been marked as Modified for some period of time. I believe that most of these issues have been fixed, so I'm resolving these bugs as Rawhide. If the bug you are seeing still exists, please reopen this report and mark it as Reopened.