I vote for simplicity (almost always) until the need for something more complex has been proven.

On 05-Sep-13 00:05, Jack Hennessey wrote:

Taking into account what has been said so far, here are some examples if we just extend ProductSearch.

Here we just add an "UpgradeCode" attribute:
<!--Enumerates related products and returns the highest version found.-->
<ProductSearch UpgradeCode="{SOME-GUID}" Variable="AcmeProductVersion" />

Rob mentioned returning all installed languages but sounds like Bob is possibly on the fence for that one(?). However, if that were to be implemented, adding a "Return" {highestVersion, all} attribute could allow this authoring: <!--Enumerates related products and returns a semicolon delimited list of all languages found.--> <ProductSearch UpgradeCode="{SOME-GUID}" Result="language" Return="all" Variable="AcmeProductLanguage" />

If you just wanted the language of the most recent version then omit the "Return" attribute as the default would be "highestVersion": <!--Enumerates related products and returns the language of the highest version found.--> <ProductSearch UpgradeCode="{SOME-GUID}" Result="language" Variable="AcmeProductLanguages" />

And of course if you want to get all versions:
<!--Enumerates related products and returns all versions found.-->
<ProductSearch UpgradeCode="{SOME-GUID}" Return="all" Variable="AcmeProductVersions" />

As Bob mentioned, adding a "ProductCode" attribute would make things clearer but "Guid" would still be supported and would always infer a ProductCode.

Thoughts / Opinions?
Is it worth the extra complexity to return all results for related products?


On Wed, Sep 4, 2013 at 4:53 PM, Bob Arnson <b...@joyofsetup.com <mailto:b...@joyofsetup.com>> wrote:

    Actually, it's +1 on new UpgradeCode and ProductCode attributes to
    ProductSearch rather than adding an attribute that adds a mode to
    overload another attribute. (In WiX v3.x, we'd have to keep the
    existing attributes working as they do today, of course.)

    I'd like to approach this with as little functionality as needed.
    I like the idea of returning the highest-versioned product as a
    version. That feeds right into DetectCondition and
    InstallCondition expressions.

    I'm not sure the alternatives I suggested have much value and
    adding all the FindRelatedProducts goop seems like overkill.



    On 04-Sep-13 16:42, Rob Mensching wrote:
    1. Having had many +1's (and many more -1's <smile/>) thrown my
    way, my guess is Bob was voting for a new element.
    2. "Detect all installed language packages". I'm certain if we
    don't do this correctly we'll get bit somewhere.
    3. Agreed. This seems to go along with Bob's suggestion to have a
    "first, all, highest" enum to control the return.
    Honestly, these discussions are usually easier if we're
    discussing real examples. Can you throw out some XML examples of
    what you're thinking that might address all the feedback and we
    can tweak from there?


    On Wed, Sep 4, 2013 at 10:38 AM, Jack Hennessey
    <jhennessey5...@gmail.com <mailto:jhennessey5...@gmail.com>> wrote:

        1. Was your +1 for using a new attribute in the ProductSearch
        element or for creating a new element altogether?

        2. I agree that most of the time you are just looking for the
        latest version so you can do a comparison. I am having
        trouble thinking of a use case for returning all related
        products but I'm sure one exists...does anyone have one?

        3. I like how ProductSearch lets you get at other information
        as well (and can see it being extended in the future to
        support more of the stuff here:
        
http://msdn.microsoft.com/en-us/library/windows/desktop/aa370130%28v=vs.85%29.aspx
        ). That is why I think it would be nice to keep things in one
        element if possible.



        On Fri, Aug 30, 2013 at 11:19 PM, Bob Arnson
        <b...@joyofsetup.com <mailto:b...@joyofsetup.com>> wrote:

            On 30-Aug-13 18:14, Rob Mensching wrote:
            > 1. What about using a separate attribute or maybe a new
            element? It
            > might be good to show a few examples to see which works
            best.
            > Especially since UpgradeCode can actually find multiple
            products.
            +1

            > 2. What if you find several products that have the same
            version
            > number? It'd almost be better to return all
            ProductCodes that match.
            > This is why I suggested a new element may be better,
            since there may
            > end up many ways to slice the results. <sigh/>
            > This feature was never terribly straight forward
            because the result is
            > an Array of Guids, not a single Guid, and Burn doesn't
            support Arrays.
            > <smile/>
            80 percent case (90 maybe?) would be covered by a boolean
            result with a
            min-version check. But maybe another Result enum for
            boolean, first
            match, all matches semicolon delimited.

            --
            sig://boB
            http://joyofsetup.com/


            
------------------------------------------------------------------------------
            Learn the latest--Visual Studio 2012, SharePoint 2013,
            SQL 2012, more!
            Discover the easy way to master current and previous
            Microsoft technologies
            and advance your career. Get an incredible 1,500+ hours
            of step-by-step
            tutorial videos with LearnDevNow. Subscribe today and save!
            
http://pubads.g.doubleclick.net/gampad/clk?id=58040911&iu=/4140/ostg.clktrk
            _______________________________________________
            WiX-devs mailing list
            WiX-devs@lists.sourceforge.net
            <mailto:WiX-devs@lists.sourceforge.net>
            https://lists.sourceforge.net/lists/listinfo/wix-devs



        
------------------------------------------------------------------------------
        Learn the latest--Visual Studio 2012, SharePoint 2013, SQL
        2012, more!
        Discover the easy way to master current and previous
        Microsoft technologies
        and advance your career. Get an incredible 1,500+ hours of
        step-by-step
        tutorial videos with LearnDevNow. Subscribe today and save!
        
http://pubads.g.doubleclick.net/gampad/clk?id=58040911&iu=/4140/ostg.clktrk
        _______________________________________________
        WiX-devs mailing list
        WiX-devs@lists.sourceforge.net
        <mailto:WiX-devs@lists.sourceforge.net>
        https://lists.sourceforge.net/lists/listinfo/wix-devs




    
------------------------------------------------------------------------------
    Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012,
    more! Discover the easy way to master current and previous
    Microsoft technologies and advance your career. Get an incredible
    1,500+ hours of step-by-step tutorial videos with LearnDevNow.
    Subscribe today and save!
    http://pubads.g.doubleclick.net/gampad/clk?id=58041391&iu=/4140/ostg.clktrk


    _______________________________________________
    WiX-devs mailing list
    WiX-devs@lists.sourceforge.net  <mailto:WiX-devs@lists.sourceforge.net>
    https://lists.sourceforge.net/lists/listinfo/wix-devs

-- sig://boB
    http://joyofsetup.com/


    
------------------------------------------------------------------------------
    Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more!
    Discover the easy way to master current and previous Microsoft
    technologies
    and advance your career. Get an incredible 1,500+ hours of
    step-by-step
    tutorial videos with LearnDevNow. Subscribe today and save!
    http://pubads.g.doubleclick.net/gampad/clk?id=58041391&iu=/4140/ostg.clktrk
    _______________________________________________
    WiX-devs mailing list
    WiX-devs@lists.sourceforge.net <mailto:WiX-devs@lists.sourceforge.net>
    https://lists.sourceforge.net/lists/listinfo/wix-devs




------------------------------------------------------------------------------
Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more!
Discover the easy way to master current and previous Microsoft technologies
and advance your career. Get an incredible 1,500+ hours of step-by-step
tutorial videos with LearnDevNow. Subscribe today and save!
http://pubads.g.doubleclick.net/gampad/clk?id=58041391&iu=/4140/ostg.clktrk


_______________________________________________
WiX-devs mailing list
WiX-devs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wix-devs

--
sig://boB
http://joyofsetup.com/

------------------------------------------------------------------------------
How ServiceNow helps IT people transform IT departments:
1. Consolidate legacy IT systems to a single system of record for IT
2. Standardize and globalize service processes across IT
3. Implement zero-touch automation to replace manual, redundant tasks
http://pubads.g.doubleclick.net/gampad/clk?id=51271111&iu=/4140/ostg.clktrk
_______________________________________________
WiX-devs mailing list
WiX-devs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wix-devs

Reply via email to