Mailinglist Archive: zypp-devel (92 mails)

< Previous Next >
Re: [zypp-devel] Product type handling
  • From: Michael Andres <ma@xxxxxxx>
  • Date: Fri, 3 Aug 2007 20:42:28 +0200
  • Message-id: <20070803184228.GA1183@xxxxxxx>
On Thu, Jul 19, Jiri Srain wrote:

> As I stated today during the meeting, please, update following way the source 
> parsers in ZYPP:
> 
> For SuSEtags add to the content file
> 
> TYPE base

In libzypp-3.13.11:

- adapted the SuSEtags content file parser to parse

        TYPE <stringvalue>

- Fixed CacheStore to write the attribute, and ProductImpl to
  provide it.

- Product interface: added

        /** Get the product type (base, add-on) */
        std::string type() const;
        
  and deprecated category().

 
> For YUM, we already have in the metadata definition
> 
>     attribute type { text },
> 
> which should have values "base" or "add-on"; the parser itself has nandling 
> this attribute commented out.

- In parser/yum/ProductFileReader.cc:

        -        // TODO what's this?
        -        // _product->? = reader_r->getAttribute("type").asString();
        +        // product type (base, add-on)
        +        _product->type = reader_r->getAttribute("type").asString();

  Hope that's correct. Ask Jano to have a look.


> The zypp::Product::category should return the type of the product. Since we 
> broke even source-code compatibility of the library as whole anyway, it may 
> make sense to rename it to zypp::Product::type to be consistent with the 
> metadata.

Done.


-- 

cu,
    Michael Andres

+------------------------------------------------------------------+
Key fingerprint = 2DFA 5D73 18B1 E7EF A862  27AC 3FB8 9E3A 27C6 B0E4
+------------------------------------------------------------------+
Michael Andres             YaST Development            ma@xxxxxxxxxx
SUSE LINUX Products GmbH, GF:  Markus Rex,  HRB 16746 (AG Nuernberg)
Maxfeldstrasse 5, D-90409 Nuernberg, Germany, ++49 (0)911 - 740 53-0
+------------------------------------------------------------------+

-- 
To unsubscribe, e-mail: zypp-devel+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: zypp-devel+help@xxxxxxxxxxxx

< Previous Next >
This Thread
  • No further messages