diff options
author | Pascal Rigaux <pixel@mandriva.com> | 2007-12-17 09:11:52 +0000 |
---|---|---|
committer | Pascal Rigaux <pixel@mandriva.com> | 2007-12-17 09:11:52 +0000 |
commit | 24f28f296b5794f6747f43fd1c86230997c6c187 (patch) | |
tree | 369ce4269359d9f165b04ecd23afbf1347397d7d /URPM.xs | |
parent | 379df21b0ed70a5e9b22cdadaa6a5abc5f0d49df (diff) | |
download | perl-URPM-24f28f296b5794f6747f43fd1c86230997c6c187.tar perl-URPM-24f28f296b5794f6747f43fd1c86230997c6c187.tar.gz perl-URPM-24f28f296b5794f6747f43fd1c86230997c6c187.tar.bz2 perl-URPM-24f28f296b5794f6747f43fd1c86230997c6c187.tar.xz perl-URPM-24f28f296b5794f6747f43fd1c86230997c6c187.zip |
- suggests:
handle both RPMTAG_SUGGESTSNAME (as done in SuSE and in Mandriva > 2008.0)
and RPMTAG_REQUIRENAME + RPMSENSE_MISSINGOK (as done in Mandriva 2008.0)
(backport from HEAD)
Diffstat (limited to 'URPM.xs')
-rw-r--r-- | URPM.xs | 34 |
1 files changed, 19 insertions, 15 deletions
@@ -275,14 +275,14 @@ ranges_overlap(int_32 aflags, char *sa, int_32 bflags, char *sb, int b_nopromote } } -static int has_suggests; -int is_suggests(int_32 flags) { +static int has_old_suggests; +int is_old_suggests(int_32 flags) { int is = flags & RPMSENSE_MISSINGOK; - if (is) has_suggests = is; + if (is) has_old_suggests = is; return is; } -int is_not_suggests(int_32 flags) { - return !is_suggests(flags); +int is_not_old_suggests(int_32 flags) { + return !is_old_suggests(flags); } typedef int (*callback_list_str)(char *s, int slen, char *name, int_32 flags, char *evr, void *param); @@ -307,7 +307,7 @@ callback_list_str_xpush_requires(char *s, int slen, char *name, int_32 flags, ch dSP; if (s) { XPUSHs(sv_2mortal(newSVpv(s, slen))); - } else if (is_not_suggests(flags)) { + } else if (is_not_old_suggests(flags)) { char buff[4096]; int len = print_list_entry(buff, sizeof(buff)-1, name, flags, evr); if (len >= 0) @@ -318,11 +318,11 @@ callback_list_str_xpush_requires(char *s, int slen, char *name, int_32 flags, ch return 0; } static int -callback_list_str_xpush_suggests(char *s, int slen, char *name, int_32 flags, char *evr, void *param) { +callback_list_str_xpush_old_suggests(char *s, int slen, char *name, int_32 flags, char *evr, void *param) { dSP; if (s) { XPUSHs(sv_2mortal(newSVpv(s, slen))); - } else if (is_suggests(flags)) { + } else if (is_old_suggests(flags)) { char buff[4096]; int len = print_list_entry(buff, sizeof(buff)-1, name, flags, evr); if (len >= 0) @@ -460,7 +460,7 @@ xpush_simple_list_str(Header header, int_32 tag_name) { free(list); PUTBACK; return c; - } + } else return 0; } void @@ -810,10 +810,12 @@ pack_header(URPM__Package pkg) { pkg->info = memcpy(malloc(p-buff), buff, p-buff); } if (pkg->requires == NULL && pkg->suggests == NULL) - has_suggests = 0; - pkg->requires = pack_list(pkg->h, RPMTAG_REQUIRENAME, RPMTAG_REQUIREFLAGS, RPMTAG_REQUIREVERSION, is_not_suggests); - if (has_suggests) - pkg->suggests = pack_list(pkg->h, RPMTAG_REQUIRENAME, RPMTAG_REQUIREFLAGS, RPMTAG_REQUIREVERSION, is_suggests); + has_old_suggests = 0; + pkg->requires = pack_list(pkg->h, RPMTAG_REQUIRENAME, RPMTAG_REQUIREFLAGS, RPMTAG_REQUIREVERSION, is_not_old_suggests); + if (has_old_suggests) + pkg->suggests = pack_list(pkg->h, RPMTAG_REQUIRENAME, RPMTAG_REQUIREFLAGS, RPMTAG_REQUIREVERSION, is_old_suggests); + else + pkg->suggests = pack_list(pkg->h, RPMTAG_SUGGESTSNAME, 0, 0, NULL); if (pkg->obsoletes == NULL) pkg->obsoletes = pack_list(pkg->h, RPMTAG_OBSOLETENAME, RPMTAG_OBSOLETEFLAGS, RPMTAG_OBSOLETEVERSION, NULL); if (pkg->conflicts == NULL) @@ -2057,8 +2059,10 @@ Pkg_suggests(pkg) URPM::Package pkg PPCODE: PUTBACK; - return_list_str(pkg->suggests, pkg->h, RPMTAG_REQUIRENAME, RPMTAG_REQUIREFLAGS, 0, - callback_list_str_xpush_suggests, NULL); + int count = return_list_str(pkg->suggests, pkg->h, RPMTAG_SUGGESTSNAME, 0, 0, callback_list_str_xpush, NULL); + if (count == 0) + return_list_str(pkg->suggests, pkg->h, RPMTAG_REQUIRENAME, RPMTAG_REQUIREFLAGS, 0, + callback_list_str_xpush_old_suggests, NULL); SPAGAIN; void |