From sacadmin Thu Nov  5 00:40:52 2009
Received: from sac.sfbay.sun.com (localhost [127.0.0.1])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id nA58eqYI008932;
	Thu, 5 Nov 2009 00:40:52 -0800 (PST)
Received: (from xc149992@localhost)
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8/Submit) id nA58eqT3008928;
	Thu, 5 Nov 2009 00:40:52 -0800 (PST)
Date: Thu, 5 Nov 2009 00:40:52 -0800 (PST)
From: Xiang-Dong Frank Che <xc149992@sac.sfbay.sun.com>
Message-Id: <200911050840.nA58eqT3008928@sac.sfbay.sun.com>
To: PSARC-record@sac.sfbay.sun.com
Cc: frank.che@sun.com
Subject: Add Text Mode UI and Common Library to Device Driver Utility [PSARC/2009/602 Self Review]
Status: RO
Content-Length: 598


Template Version: @(#)sac_nextcase 1.68 02/23/09 SMI
This information is Copyright 2009 Sun Microsystems
1. Introduction
    1.1. Project/Component Working Name:
	 Add Text Mode UI and Common Library to Device Driver Utility
    1.2. Name of Document Author/Supplier:
	 Author:  Bill Yan
    1.3  Date of This Document:
	05 November, 2009
4. Technical Description
    See the case directory for more detail

6. Resources and Schedule
    6.4. Steering Committee requested information
   	6.4.1. Consolidation C-team Name:
		unknown
    6.5. ARC review type: Automatic
    6.6. ARC Exposure: open


From Frank.Che@sun.com Thu Nov  5 21:13:36 2009
Received: from sunmail4.singapore.sun.com (sunmail4.Singapore.Sun.COM [129.158.71.19])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id nA65DZk6002798
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 5 Nov 2009 21:13:36 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by sunmail4.singapore.sun.com (8.13.4+Sun/8.13.3/ENSMAIL,v2.2) with ESMTP id nA65DSA5028116
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Fri, 6 Nov 2009 13:13:34 +0800 (SGT)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KSO0090596MJE00@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Thu, 05 Nov 2009 21:13:34 -0800 (PST)
Received: from sineb-mail-1.sun.com ([192.18.19.6])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KSO00I2B96L4FA0@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@sun.com); Thu,
 05 Nov 2009 21:13:34 -0800 (PST)
Received: from fe-apac-06.sun.com
 (fe-apac-06.sun.com [192.18.19.177] (may be forged))
	by sineb-mail-1.sun.com (8.13.6+Sun/8.12.9) with ESMTP id nA65DVf1020396	for
 <PSARC-ext@sun.com>; Fri, 06 Nov 2009 05:13:32 +0000 (GMT)
Received: from conversion-daemon.mail-apac.sun.com by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KSO0070093MML00@mail-apac.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Fri, 06 Nov 2009 13:13:31 +0800 (SGT)
Received: from [129.158.218.60] ([unknown] [129.158.218.60])
 by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KSO005VP96BDHD0@mail-apac.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Fri, 06 Nov 2009 13:13:24 +0800 (SGT)
Date: Fri, 06 Nov 2009 13:08:48 +0800
From: Frank Che <Frank.Che@sun.com>
Subject: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 Self Review]
Sender: Frank.Che@sun.com
To: PSARC-ext@sun.com
Cc: Bill Yan <Xue-Yang.Yan@sun.com>
Message-id: <4AF3AF60.7050801@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
User-Agent: Thunderbird 2.0.0.19 (X11/20090218)
Status: RO
Content-Length: 4794

I'm sponsoring this case for Bill Yan.  This project is to add several 
more functions to the OpenSolaris Device Drive Utility tool. A Micro 
release binding is proposed.

I have marked it as "closed approved automatic" since the change is 
straightforward and the interface is clear. If there is request, I'd be 
happy to upgrade it to a fast track and set the timer.

-Frank


4. Technical Description

4.1 Summary

   The intent of this project is to make the following enhancements to 
Device
   Driver Utility Tool [1]:
   * Add text mode UI to Device Driver Utility Tool.
   * Provide a common library to check missing driver.
   * Enhance driver installation capability.

   Requested release binding is Micro.

4.2 Details

   This project propose to make the following enhancements to Device Driver
   Utility Tool [1]:

4.2.1 Add text mode UI to Device Driver Utility Tool
   Current Device Driver Utility only has a graphical UI. In order to 
meet the
   requirement from OpenSolaris Text mode installer, a text mode UI is
   proposed to add to the Device Driver Utility. This text mode UI is 
based on
   ncurse, and only lists device(s) that doesn't have driver. This text 
mode
   UI also enables the user to install missing driver(s).

4.2.2 Provide a common library to check missing driver
   Current Device Driver Utility mixed core functions and UI together, this
   implementation may bring trouble to future expansion of this tool. In 
order
   to make structure of this tool better, Device Driver Utility will 
provide a
   common library to check missing driver. This common library will serve
   Device Driver Utility and/or other program. The APIs privided by this
   library is available in "Exported Interfaces" in this document. Detail
   descriptions of each API is available in "DDU Programming Interface
   Specification" [2] document.

4.2.3 Enhance driver installation capability
   Current Device Driver Utility only installs driver from IPS 
repository. In
   case the driver supplier provides driver in other format, or network
   connection is not available(for example, network controller driver is
   missing), the user need to install driver(s) in other format. In this
   project, Device Driver Utility will be enhanced to be able to install
   driver(s) in P5I [3], DU-Image [4] and SVR4 [5] formats via call 
"pkg" and
   "pkgadd" commands.

4.3 Imported Interfaces

   Interface         |  Stability  | Description
   
------------------+-------------+----------------------------------------
   _curses.so        | Committed   | PSARC/1996/219
   
------------------+-------------+----------------------------------------
   _curses_panel.so  | Committed   | PSARC/1996/219
   
------------------+-------------+----------------------------------------
   pkgadd            | Committed   | PSARC/1995/160
   
------------------+-------------+----------------------------------------
   pkg               | Uncommitted | PSARC/2008/190
   
------------------+-------------+----------------------------------------

4.4 Exported Interfaces

   Interface               |  Stability  | Description
   
------------------------+-------------+----------------------------------
   /usr/ddu/ddu-text.py    | Uncommitted | Command to invoke text mode UI
   
------------------------+-------------+----------------------------------
   ddu_devscan()           | Uncommitted | DDU library API [2]
   
------------------------+-------------+----------------------------------
   ddu_package_lookup()    | Uncommitted | DDU library API [2]
   ddu_build_repo_list()   |             |
   ddu_install_package()   |             |
   
------------------------+-------------+----------------------------------
   ddu_repo_object         | Uncommitted | DDU library object [2]
   ddu_package_object      |             |
   
------------------------+-------------+----------------------------------
   ddu_error_message()     | Uncommitted | DDU error message library API 
[2]
   
------------------------+-------------+----------------------------------


4.5 References
   [1] Device Driver Utility 1.1 PSARC Case:
   http://sac.sfbay/PSARC/2008/504/
   [2] DDU Programming Interface Specification:
   
http://www.opensolaris.org/os/project/caiman/Driver_Update/ddu-prog-iface-spec.txt 

   [3] P5I: File format used by pkg(5) to fully identify a package 
(including
   name and location) or group of packages.
   [4] DU-Image: Driver Update Image. A legacy format used to deploy driver
   updates, used by Install Time Update.
   [5] SVR4: Package format used for Solaris 10 and before.

6. Resources and Schedule
   6.4 Steering Committee Requested Information
   6.4.1 Consolidation C-team Name: None
   6.5 ARC review type: Self Review
   6.6 ARC exposure: Open




From Frank.Che@Sun.COM Thu Feb 25 22:17:45 2010
Received: from sunmail2sca.sfbay.sun.com (sunmail2sca.SFBay.Sun.COM [129.145.155.234])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o1Q6HjJg018324
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 25 Feb 2010 22:17:45 -0800 (PST)
Received: from nwk-avmta-2.sfbay.sun.com (nwk-avmta-2.SFBay.Sun.COM [129.145.155.6])
	by sunmail2sca.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o1Q6Hjn6019280
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 25 Feb 2010 22:17:45 -0800 (PST)
Received: from pmxchannel-daemon.nwk-avmta-2.sfbay.sun.com by
 nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYF00501QTLX500@nwk-avmta-2.sfbay.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Thu, 25 Feb 2010 22:17:45 -0800 (PST)
Received: from sineb-mail-1.sun.com ([192.18.19.6])
 by nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYF00F6QQTJI1D0@nwk-avmta-2.sfbay.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@sun.com); Thu,
 25 Feb 2010 22:17:44 -0800 (PST)
Received: from fe-apac-05.sun.com
 (fe-apac-05.sun.com [192.18.19.176] (may be forged))
	by sineb-mail-1.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o1Q6HhE3026059	for
 <PSARC-ext@sun.com>; Fri, 26 Feb 2010 06:17:43 +0000 (GMT)
Received: from conversion-daemon.mail-apac.sun.com by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYF00F00QPLJE00@mail-apac.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Fri, 26 Feb 2010 14:17:43 +0800 (SGT)
Received: from [129.158.218.60] ([unknown] [129.158.218.60])
 by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYF005G2QTICKB0@mail-apac.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Fri, 26 Feb 2010 14:17:43 +0800 (SGT)
Date: Fri, 26 Feb 2010 14:17:37 +0800
From: Frank Che <Frank.Che@Sun.COM>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4AF3AF60.7050801@sun.com>
Sender: Frank.Che@Sun.COM
To: PSARC-ext@Sun.COM
Cc: Bill Yan <Xue-Yang.Yan@Sun.COM>, Jack.A.Schwartz@Sun.COM
Message-id: <4B876781.4090007@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com>
User-Agent: Thunderbird 2.0.0.19 (X11/20090218)
Status: RO
Content-Length: 5721

I'm reopening this case as a fast track, as requested by the project 
team. The timer is set for 03/05/2010. Note: this project is not 
integrated yet.

I have saved the new spec as spec.txt in the case directory. Compared 
with the previous case spec, the following changes were made:
1. The ddu_error_message interface in previous spec was removed.
2. A new interface was introduced: ddu_dev_data

The DDU Programming Interface Specification document, as part of the 
case material to be reviewed, was also saved in the case directory.

Please also be noted that while this project currently doesn't not 
belong to any consolidation, the project team is working actively to 
integrate it into the ON consolidation.

Thanks,
Frank.

Frank Che wrote:
> I'm sponsoring this case for Bill Yan.  This project is to add several 
> more functions to the OpenSolaris Device Drive Utility tool. A Micro 
> release binding is proposed.
>
> I have marked it as "closed approved automatic" since the change is 
> straightforward and the interface is clear. If there is request, I'd 
> be happy to upgrade it to a fast track and set the timer.
>
> -Frank
>
>
> 4. Technical Description
>
> 4.1 Summary
>
>   The intent of this project is to make the following enhancements to 
> Device
>   Driver Utility Tool [1]:
>   * Add text mode UI to Device Driver Utility Tool.
>   * Provide a common library to check missing driver.
>   * Enhance driver installation capability.
>
>   Requested release binding is Micro.
>
> 4.2 Details
>
>   This project propose to make the following enhancements to Device 
> Driver
>   Utility Tool [1]:
>
> 4.2.1 Add text mode UI to Device Driver Utility Tool
>   Current Device Driver Utility only has a graphical UI. In order to 
> meet the
>   requirement from OpenSolaris Text mode installer, a text mode UI is
>   proposed to add to the Device Driver Utility. This text mode UI is 
> based on
>   ncurse, and only lists device(s) that doesn't have driver. This text 
> mode
>   UI also enables the user to install missing driver(s).
>
> 4.2.2 Provide a common library to check missing driver
>   Current Device Driver Utility mixed core functions and UI together, 
> this
>   implementation may bring trouble to future expansion of this tool. 
> In order
>   to make structure of this tool better, Device Driver Utility will 
> provide a
>   common library to check missing driver. This common library will serve
>   Device Driver Utility and/or other program. The APIs privided by this
>   library is available in "Exported Interfaces" in this document. Detail
>   descriptions of each API is available in "DDU Programming Interface
>   Specification" [2] document.
>
> 4.2.3 Enhance driver installation capability
>   Current Device Driver Utility only installs driver from IPS 
> repository. In
>   case the driver supplier provides driver in other format, or network
>   connection is not available(for example, network controller driver is
>   missing), the user need to install driver(s) in other format. In this
>   project, Device Driver Utility will be enhanced to be able to install
>   driver(s) in P5I [3], DU-Image [4] and SVR4 [5] formats via call 
> "pkg" and
>   "pkgadd" commands.
>
> 4.3 Imported Interfaces
>
>   Interface         |  Stability  | Description
>   
> ------------------+-------------+----------------------------------------
>   _curses.so        | Committed   | PSARC/1996/219
>   
> ------------------+-------------+----------------------------------------
>   _curses_panel.so  | Committed   | PSARC/1996/219
>   
> ------------------+-------------+----------------------------------------
>   pkgadd            | Committed   | PSARC/1995/160
>   
> ------------------+-------------+----------------------------------------
>   pkg               | Uncommitted | PSARC/2008/190
>   
> ------------------+-------------+----------------------------------------
>
> 4.4 Exported Interfaces
>
>   Interface               |  Stability  | Description
>   
> ------------------------+-------------+----------------------------------
>   /usr/ddu/ddu-text.py    | Uncommitted | Command to invoke text mode UI
>   
> ------------------------+-------------+----------------------------------
>   ddu_devscan()           | Uncommitted | DDU library API [2]
>   
> ------------------------+-------------+----------------------------------
>   ddu_package_lookup()    | Uncommitted | DDU library API [2]
>   ddu_build_repo_list()   |             |
>   ddu_install_package()   |             |
>   
> ------------------------+-------------+----------------------------------
>   ddu_repo_object         | Uncommitted | DDU library object [2]
>   ddu_package_object      |             |
>   
> ------------------------+-------------+----------------------------------
>   ddu_error_message()     | Uncommitted | DDU error message library 
> API [2]
>   
> ------------------------+-------------+----------------------------------
>
>
> 4.5 References
>   [1] Device Driver Utility 1.1 PSARC Case:
>   http://sac.sfbay/PSARC/2008/504/
>   [2] DDU Programming Interface Specification:
>   
> http://www.opensolaris.org/os/project/caiman/Driver_Update/ddu-prog-iface-spec.txt 
>
>   [3] P5I: File format used by pkg(5) to fully identify a package 
> (including
>   name and location) or group of packages.
>   [4] DU-Image: Driver Update Image. A legacy format used to deploy 
> driver
>   updates, used by Install Time Update.
>   [5] SVR4: Package format used for Solaris 10 and before.
>
> 6. Resources and Schedule
>   6.4 Steering Committee Requested Information
>   6.4.1 Consolidation C-team Name: None
>   6.5 ARC review type: Self Review
>   6.6 ARC exposure: Open
>
>
>

From Sebastien.Roy@sun.com Wed Mar  3 07:28:56 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o23FStdx023192
	for <psarc-ext@sac.sfbay.sun.com>; Wed, 3 Mar 2010 07:28:55 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o23FSo3e027685
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Wed, 3 Mar 2010 08:28:55 -0700 (MST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYP00M0NPO7LA00@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Wed, 03 Mar 2010 07:28:55 -0800 (PST)
Received: from brmea-mail-1.sun.com ([192.18.98.31])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYP00EDSPO6NK30@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Wed,
 03 Mar 2010 07:28:55 -0800 (PST)
Received: from fe-amer-09.sun.com ([192.18.109.79])
	by brmea-mail-1.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o23FSssD016045	for
 <PSARC-ext@Sun.COM>; Wed, 03 Mar 2010 15:28:54 +0000 (GMT)
Received: from conversion-daemon.mail-amer.sun.com by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYP00600PKMIZ00@mail-amer.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Wed, 03 Mar 2010 08:28:54 -0700 (MST)
Received: from [129.148.174.103] ([unknown] [129.148.174.103])
 by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYP003Q8PO4GZ10@mail-amer.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Wed, 03 Mar 2010 08:28:53 -0700 (MST)
Date: Wed, 03 Mar 2010 10:26:10 -0500
From: Sebastien Roy <Sebastien.Roy@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B876781.4090007@sun.com>
Sender: Sebastien.Roy@sun.com
To: Frank Che <Frank.Che@sun.com>
Cc: PSARC-ext@sun.com, Bill Yan <Xue-Yang.Yan@sun.com>,
        Jack.A.Schwartz@sun.com
Message-id: <4B8E7F92.6050605@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100131
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 2449

On 02/26/10 01:17 AM, Frank Che wrote:
>> 4.2.3 Enhance driver installation capability
>> Current Device Driver Utility only installs driver from IPS
>> repository. In
>> case the driver supplier provides driver in other format, or network
>> connection is not available(for example, network controller driver is
>> missing), the user need to install driver(s) in other format. In this
>> project, Device Driver Utility will be enhanced to be able to install
>> driver(s) in P5I [3], DU-Image [4] and SVR4 [5] formats via call "pkg"
>> and
>> "pkgadd" commands.

Has the project team been in contact with the pkg team to ensure that 
drivers installed using this tool won't be automatically removed from 
their installed locations during pkg image-update?

> 4.4 Exported Interfaces
>
>     Interface               |  Stability  | Description
>     ------------------------+-------------+----------------------------------
>     /usr/ddu/ddu-text.py    | Uncommitted | Command to invoke text mode UI

This doesn't seem right.  There's already a /usr/bin/ddu command, and 
that appears to be the public interface (not the /usr/ddu/ddu.py script 
to which the /usr/bin/ddu symbolic link points to).  Isn't 
/usr/ddu/ddu-text.py actually a Project Private interface?

>     ------------------------+-------------+----------------------------------
>     ddu_devscan()           | Uncommitted | DDU library API [2]
>     ------------------------+-------------+----------------------------------
>     ddu_package_lookup()    | Uncommitted | DDU library API [2]
>     ddu_build_repo_list()   |             |
>     ddu_install_package()   |             |
>     ------------------------+-------------+----------------------------------
>     ddu_repo_object         | Uncommitted | DDU library object [2]
>     ddu_package_object      |             |
>     ddu_dev_data            |             |
>     ------------------------+-------------+----------------------------------

What about the library itself?  Is it a C library, a python module, 
something else...?

Based on the materials, I don't understand the semantics of /usr/bin/ddu 
--silent.  Basically, I don't grok section 2.1 of your "programming 
interface spec" (and it seems to be more than a programming interface 
spec).  You explain when --silent is used, but not what it does.  This 
seems central to the proposal, and I'd like to understand this.  A 
ddu(1M) man page would help.

-Seb

From Jack.A.Schwartz@Sun.COM Wed Mar  3 13:26:44 2010
Received: from sunmail6brm.central.sun.com (sunmail6brm.Central.Sun.COM [129.147.4.169])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o23LQiF0007108
	for <psarc-ext@sac.sfbay.sun.com>; Wed, 3 Mar 2010 13:26:44 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by sunmail6brm.central.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o23LQiHG009372
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Wed, 3 Mar 2010 15:26:44 -0600 (CST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYQ0080P68JQH00@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Wed, 03 Mar 2010 13:26:43 -0800 (PST)
Received: from sca-es-mail-2.sun.com ([192.18.43.133])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYQ00GT468FFOB0@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Wed,
 03 Mar 2010 13:26:43 -0800 (PST)
Received: from fe-sfbay-09.sun.com ([192.18.43.129])
	by sca-es-mail-2.sun.com (8.13.7+Sun/8.12.9) with ESMTP id o23LQduF007199	for
 <PSARC-ext@Sun.COM>; Wed, 03 Mar 2010 13:26:39 -0800 (PST)
Received: from conversion-daemon.fe-sfbay-09.sun.com by fe-sfbay-09.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYQ000006141H00@fe-sfbay-09.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Wed, 03 Mar 2010 13:26:39 -0800 (PST)
Received: from [129.146.226.171] ([unknown] [129.146.226.171])
 by fe-sfbay-09.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYQ00IVQ682SG90@fe-sfbay-09.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Wed, 03 Mar 2010 13:26:27 -0800 (PST)
Date: Wed, 03 Mar 2010 13:25:27 -0800
From: Jack Schwartz <Jack.A.Schwartz@Sun.COM>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8E7F92.6050605@sun.com>
Sender: Jack.A.Schwartz@Sun.COM
To: Sebastien Roy <Sebastien.Roy@Sun.COM>
Cc: Frank Che <Frank.Che@Sun.COM>, PSARC-ext@Sun.COM,
        Bill Yan <Xue-Yang.Yan@Sun.COM>
Message-id: <4B8ED3C7.5010601@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.5) Gecko/20100103
 Thunderbird/3.0
Status: RO
Content-Length: 4060

Hi Sebastien.

While I am not part of the DDU team, I have been working closely with 
them since my project depends heavily on their product.  I worked with 
them on forging the specification.

I'll answer the questions I know about and leave the others.

On 03/03/10 07:26, Sebastien Roy wrote:
> On 02/26/10 01:17 AM, Frank Che wrote:
>>> 4.2.3 Enhance driver installation capability
>>> Current Device Driver Utility only installs driver from IPS
>>> repository. In
>>> case the driver supplier provides driver in other format, or network
>>> connection is not available(for example, network controller driver is
>>> missing), the user need to install driver(s) in other format. In this
>>> project, Device Driver Utility will be enhanced to be able to install
>>> driver(s) in P5I [3], DU-Image [4] and SVR4 [5] formats via call "pkg"
>>> and
>>> "pkgadd" commands.
>
> Has the project team been in contact with the pkg team to ensure that 
> drivers installed using this tool won't be automatically removed from 
> their installed locations during pkg image-update?
If SVR4 packages are added and their files are orthogonal to IPS 
packages, then IPS will leave those files alone.  DDU will use the pkg 
command to install IPS packages.
>> 4.4 Exported Interfaces
>>
>>     Interface               |  Stability  | Description
>>     
>> ------------------------+-------------+---------------------------------- 
>>
>>     /usr/ddu/ddu-text.py    | Uncommitted | Command to invoke text 
>> mode UI
>
> This doesn't seem right.  There's already a /usr/bin/ddu command, and 
> that appears to be the public interface (not the /usr/ddu/ddu.py 
> script to which the /usr/bin/ddu symbolic link points to).  Isn't 
> /usr/ddu/ddu-text.py actually a Project Private interface?
I'll let DDU team address this.
>
>>     
>> ------------------------+-------------+---------------------------------- 
>>
>>     ddu_devscan()           | Uncommitted | DDU library API [2]
>>     
>> ------------------------+-------------+---------------------------------- 
>>
>>     ddu_package_lookup()    | Uncommitted | DDU library API [2]
>>     ddu_build_repo_list()   |             |
>>     ddu_install_package()   |             |
>>     
>> ------------------------+-------------+---------------------------------- 
>>
>>     ddu_repo_object         | Uncommitted | DDU library object [2]
>>     ddu_package_object      |             |
>>     ddu_dev_data            |             |
>>     
>> ------------------------+-------------+---------------------------------- 
>>
>
> What about the library itself?  Is it a C library, a python module, 
> something else...?
The library interfaces are implemented in python.  The interface modules 
call other python and shell scripts, and C programs, all delivered as 
part of the library.
>
> Based on the materials, I don't understand the semantics of 
> /usr/bin/ddu --silent.  Basically, I don't grok section 2.1 of your 
> "programming interface spec" (and it seems to be more than a 
> programming interface spec).  You explain when --silent is used, but 
> not what it does.  This seems central to the proposal, and I'd like to 
> understand this. 
The programming interface spec was written to complement the functional 
interface spec for the Driver Update project.  The latter shows the 
higher-level functionality and addresses what silent mode is.  I suggest 
the functional interface spec for Driver Update be moved into the case 
materials as well, for added context.

For now, it is here:
http://hub.opensolaris.org/bin/download/Project+caiman/Driver_Update/du%2Dfunc%2Dspec.txt

The GUI DDU is invoked in silent mode on the live CD, to check for 
missing drivers and install them in the background.  If additional 
drivers are still missing, the DDU pops up a window to tell the user, 
and to offer to display the GUI for manual action.

Please see section 1.5 and 3.1 of that document for details on silent mode.
> A ddu(1M) man page would help.
I'll let the DDU team address this.

     Thanks,
     Jack
>
> -Seb


From swalker@opensolaris.org Wed Mar  3 14:44:04 2010
Received: from sunmail2sca.sfbay.sun.com (sunmail2sca.SFBay.Sun.COM [129.145.155.234])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o23Mi34v015062
	for <psarc-ext@sac.sfbay.sun.com>; Wed, 3 Mar 2010 14:44:03 -0800 (PST)
Received: from nwk-avmta-2.sfbay.sun.com (nwk-avmta-2.SFBay.Sun.COM [129.145.155.6])
	by sunmail2sca.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o23Mi0pU012603;
	Wed, 3 Mar 2010 14:44:01 -0800 (PST)
Received: from pmxchannel-daemon.nwk-avmta-2.sfbay.sun.com by
 nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYQ00K059TC4400@nwk-avmta-2.sfbay.sun.com>; Wed,
 03 Mar 2010 14:44:00 -0800 (PST)
Received: from jurassic.Eng.Sun.COM ([129.146.17.59])
 by nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYQ0091Z9TCS2F0@nwk-avmta-2.sfbay.sun.com>; Wed,
 03 Mar 2010 14:44:00 -0800 (PST)
Received: from [10.7.250.123]
 (punchin-client-10-7-250-123.SFBay.Sun.COM [10.7.250.123])
	by jurassic.Eng.Sun.COM (8.14.4+Sun/8.14.4) with ESMTP id o23Mhx2h817789; Wed,
 03 Mar 2010 14:44:00 -0800 (PST)
Date: Wed, 03 Mar 2010 16:43:59 -0600
From: Shawn Walker <swalker@opensolaris.org>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8ED3C7.5010601@sun.com>
To: Jack Schwartz <Jack.A.Schwartz@sun.com>
Cc: Sebastien Roy <sebastien.roy@sun.com>, PSARC-ext@sun.com,
        Bill Yan <Xue-Yang.Yan@sun.com>, Frank Che <Frank.Che@sun.com>
Message-id: <4B8EE62F.7010905@opensolaris.org>
MIME-version: 1.0
Content-type: text/plain; charset=UTF-8; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100131
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 1560

On 03/ 3/10 03:25 PM, Jack Schwartz wrote:
> Hi Sebastien.
>
> While I am not part of the DDU team, I have been working closely with
> them since my project depends heavily on their product. I worked with
> them on forging the specification.
>
> I'll answer the questions I know about and leave the others.
>
> On 03/03/10 07:26, Sebastien Roy wrote:
>> On 02/26/10 01:17 AM, Frank Che wrote:
>>>> 4.2.3 Enhance driver installation capability
>>>> Current Device Driver Utility only installs driver from IPS
>>>> repository. In
>>>> case the driver supplier provides driver in other format, or network
>>>> connection is not available(for example, network controller driver is
>>>> missing), the user need to install driver(s) in other format. In this
>>>> project, Device Driver Utility will be enhanced to be able to install
>>>> driver(s) in P5I [3], DU-Image [4] and SVR4 [5] formats via call "pkg"
>>>> and
>>>> "pkgadd" commands.
>>
>> Has the project team been in contact with the pkg team to ensure that
>> drivers installed using this tool won't be automatically removed from
>> their installed locations during pkg image-update?
> If SVR4 packages are added and their files are orthogonal to IPS
> packages, then IPS will leave those files alone. DDU will use the pkg
> command to install IPS packages.

That isn't quite true.  If those files are installed into directories 
that belong to pkg(5) packages, they may be moved out of the way during 
uninstall, install, or update operations (salvaged to /var/pkg/lost+found).

Cheers,
-- 
Shawn Walker

From Xue-Yang.Yan@sun.com Thu Mar  4 01:14:45 2010
Received: from sunmail3mpk.sfbay.sun.com (sunmail3mpk.SFBay.Sun.COM [129.146.11.52])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o249EjL7010147
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 01:14:45 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by sunmail3mpk.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o249EijR022333
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 4 Mar 2010 01:14:44 -0800 (PST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYR00E1H30KTF00@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 01:14:44 -0800 (PST)
Received: from sineb-mail-2.sun.com ([192.18.19.7])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYR00LZ630I1IE0@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 01:14:43 -0800 (PST)
Received: from fe-apac-06.sun.com
 (fe-apac-06.sun.com [192.18.19.177] (may be forged))
	by sineb-mail-2.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o249EgJY020909	for
 <PSARC-ext@Sun.COM>; Thu, 04 Mar 2010 09:14:42 +0000 (GMT)
Received: from conversion-daemon.mail-apac.sun.com by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYR00G002W3GG00@mail-apac.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 17:14:42 +0800 (SGT)
Received: from [129.158.218.53] ([unknown] [129.158.218.53])
 by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYR00EEC30G2SF0@mail-apac.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 17:14:42 +0800 (SGT)
Date: Thu, 04 Mar 2010 17:11:51 +0800
From: Bill Yan <Xue-Yang.Yan@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8ED3C7.5010601@sun.com>
Sender: Xue-Yang.Yan@sun.com
To: Sebastien Roy <Sebastien.Roy@sun.com>
Cc: Jack Schwartz <Jack.A.Schwartz@sun.com>, Frank Che <Frank.Che@sun.com>,
        PSARC-ext@sun.com,
        zhongyuan sun - Sun Microsystems - Beijing China
 <Zhongyuan.Sun@sun.com>,
        "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>
Message-id: <4B8F7957.1040401@sun.com>
MIME-version: 1.0
Content-type: text/plain; format=flowed; charset=UTF-8
Content-transfer-encoding: 8BIT
X-Accept-Language: zh-cn
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; zh-CN; rv:1.7) Gecko/20060627
Status: RO
Content-Length: 4446

Hi Sebastien,

My comments are in line. Thanks for the help.

Regards,
Bill
Jack Schwartz 已写入:

> Hi Sebastien.
>
> While I am not part of the DDU team, I have been working closely with 
> them since my project depends heavily on their product. I worked with 
> them on forging the specification.
>
> I'll answer the questions I know about and leave the others.
>
> On 03/03/10 07:26, Sebastien Roy wrote:
>
>> On 02/26/10 01:17 AM, Frank Che wrote:
>>
>>>> 4.2.3 Enhance driver installation capability
>>>> Current Device Driver Utility only installs driver from IPS
>>>> repository. In
>>>> case the driver supplier provides driver in other format, or network
>>>> connection is not available(for example, network controller driver is
>>>> missing), the user need to install driver(s) in other format. In this
>>>> project, Device Driver Utility will be enhanced to be able to install
>>>> driver(s) in P5I [3], DU-Image [4] and SVR4 [5] formats via call "pkg"
>>>> and
>>>> "pkgadd" commands.
>>>
>>
>> Has the project team been in contact with the pkg team to ensure that 
>> drivers installed using this tool won't be automatically removed from 
>> their installed locations during pkg image-update?
>
> If SVR4 packages are added and their files are orthogonal to IPS 
> packages, then IPS will leave those files alone. DDU will use the pkg 
> command to install IPS packages.
>
>>> 4.4 Exported Interfaces
>>>
>>> Interface | Stability | Description
>>> ------------------------+-------------+---------------------------------- 
>>>
>>> /usr/ddu/ddu-text.py | Uncommitted | Command to invoke text mode UI
>>
>>
>> This doesn't seem right. There's already a /usr/bin/ddu command, and 
>> that appears to be the public interface (not the /usr/ddu/ddu.py 
>> script to which the /usr/bin/ddu symbolic link points to). Isn't 
>> /usr/ddu/ddu-text.py actually a Project Private interface?
>
> I'll let DDU team address this.

ddu-text.py is not a Project Private interface, it's used by text-mode 
installer to install missing drivers during the text-mode OS installation.

>>
>>> ------------------------+-------------+---------------------------------- 
>>>
>>> ddu_devscan() | Uncommitted | DDU library API [2]
>>> ------------------------+-------------+---------------------------------- 
>>>
>>> ddu_package_lookup() | Uncommitted | DDU library API [2]
>>> ddu_build_repo_list() | |
>>> ddu_install_package() | |
>>> ------------------------+-------------+---------------------------------- 
>>>
>>> ddu_repo_object | Uncommitted | DDU library object [2]
>>> ddu_package_object | |
>>> ddu_dev_data | |
>>> ------------------------+-------------+---------------------------------- 
>>>
>>
>>
>> What about the library itself? Is it a C library, a python module, 
>> something else...?
>
> The library interfaces are implemented in python. The interface 
> modules call other python and shell scripts, and C programs, all 
> delivered as part of the library.
>
>>
>> Based on the materials, I don't understand the semantics of 
>> /usr/bin/ddu --silent. Basically, I don't grok section 2.1 of your 
>> "programming interface spec" (and it seems to be more than a 
>> programming interface spec). You explain when --silent is used, but 
>> not what it does. This seems central to the proposal, and I'd like to 
>> understand this. 
>
> The programming interface spec was written to complement the 
> functional interface spec for the Driver Update project. The latter 
> shows the higher-level functionality and addresses what silent mode 
> is. I suggest the functional interface spec for Driver Update be moved 
> into the case materials as well, for added context.
>
> For now, it is here:
> http://hub.opensolaris.org/bin/download/Project+caiman/Driver_Update/du%2Dfunc%2Dspec.txt 
>
>
> The GUI DDU is invoked in silent mode on the live CD, to check for 
> missing drivers and install them in the background. If additional 
> drivers are still missing, the DDU pops up a window to tell the user, 
> and to offer to display the GUI for manual action.
>
> Please see section 1.5 and 3.1 of that document for details on silent 
> mode.
>
>> A ddu(1M) man page would help.
>
> I'll let the DDU team address this.

ddu --silent is not a public interface, it's only used to invoke the GUI 
DDU in silent mode by GNOME autostart application. And ddu is a GUI 
application that has an on-line help.


>
> Thanks,
> Jack
>
>>
>> -Seb
>
>


From Sebastien.Roy@Sun.COM Thu Mar  4 04:58:09 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o24Cw9xp013557
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 04:58:09 -0800 (PST)
Received: from nwk-avmta-2.sfbay.sun.com (nwk-avmta-2.SFBay.Sun.COM [129.145.155.6])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o24Cw8E0000497
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 4 Mar 2010 05:58:08 -0700 (MST)
Received: from pmxchannel-daemon.nwk-avmta-2.sfbay.sun.com by
 nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYR0090BDCWTF00@nwk-avmta-2.sfbay.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 04:58:08 -0800 (PST)
Received: from brmea-mail-1.sun.com ([192.18.98.31])
 by nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYR00N07DCW0QC0@nwk-avmta-2.sfbay.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 04:58:08 -0800 (PST)
Received: from fe-amer-09.sun.com ([192.18.109.79])
	by brmea-mail-1.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o24Cw8Em013239	for
 <PSARC-ext@Sun.COM>; Thu, 04 Mar 2010 12:58:08 +0000 (GMT)
Received: from conversion-daemon.mail-amer.sun.com by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYR00J00D71PR00@mail-amer.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 05:58:08 -0700 (MST)
Received: from [192.168.1.3] ([unknown] [173.76.16.34])
 by mail-amer.sun.com (Sun Java(tm) System Messaging Server 7u2-7.04 64bit
 (built Jul  2 2009)) with ESMTPSA id <0KYR00MMMDCU7F60@mail-amer.sun.com> for
 PSARC-ext@Sun.COM (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 05:58:07 -0700 (MST)
Date: Thu, 04 Mar 2010 07:58:06 -0500
From: Sebastien Roy <Sebastien.Roy@Sun.COM>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8F7957.1040401@sun.com>
Sender: Sebastien.Roy@Sun.COM
To: Bill Yan <Xue-Yang.Yan@Sun.COM>
Cc: Jack Schwartz <Jack.A.Schwartz@Sun.COM>, Frank Che <Frank.Che@Sun.COM>,
        PSARC-ext@Sun.COM,
        zhongyuan sun - Sun Microsystems - Beijing China
 <Zhongyuan.Sun@Sun.COM>,
        "Tony.Hu@Sun.COM" <Tony.Hu@Sun.COM>
Message-id: <4B8FAE5E.4060001@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8F7957.1040401@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100131
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 1644

Hi Bill,

On 03/ 4/10 04:11 AM, Bill Yan wrote:
>>>> 4.4 Exported Interfaces
>>>>
>>>> Interface | Stability | Description
>>>> ------------------------+-------------+----------------------------------
>>>>
>>>> /usr/ddu/ddu-text.py | Uncommitted | Command to invoke text mode UI
>>>
>>>
>>> This doesn't seem right. There's already a /usr/bin/ddu command, and
>>> that appears to be the public interface (not the /usr/ddu/ddu.py
>>> script to which the /usr/bin/ddu symbolic link points to). Isn't
>>> /usr/ddu/ddu-text.py actually a Project Private interface?
>>
>> I'll let DDU team address this.
>
> ddu-text.py is not a Project Private interface, it's used by text-mode
> installer to install missing drivers during the text-mode OS installation.

You've just described a Project Private interface.  It's not used 
directly by the user; it's merely an implementation detail of the 
text-mode installer.

How does the user invoke the text-mode installer?  Whatever that is 
should be your public interface.

>>> A ddu(1M) man page would help.
>>
>> I'll let the DDU team address this.
>
> ddu --silent is not a public interface, it's only used to invoke the GUI
> DDU in silent mode by GNOME autostart application. And ddu is a GUI
> application that has an on-line help.

Okay, so what I'm asking for is a description of what --silent mode 
does.  Your materials go out of their way to state that the ddu command 
gets called with --silent but doesn't explain what that means.  Given 
your statement above regarding its application to the GUI, what does it 
have to do with a text mode UI (i.e., why is that relevant to this case?).

-Seb

From Frank.Che@sun.com Thu Mar  4 06:13:57 2010
Received: from sunmail6brm.central.sun.com (sunmail6brm.Central.Sun.COM [129.147.4.169])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o24EDvY0014668
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 06:13:57 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by sunmail6brm.central.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o24EDr8V020202
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 4 Mar 2010 08:13:57 -0600 (CST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYR00L0JGV83B00@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 06:13:56 -0800 (PST)
Received: from sineb-mail-1.sun.com ([192.18.19.6])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYR005DUGV3OV50@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 06:13:52 -0800 (PST)
Received: from fe-apac-06.sun.com
 (fe-apac-06.sun.com [192.18.19.177] (may be forged))
	by sineb-mail-1.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o24EDo4n018716	for
 <PSARC-ext@Sun.COM>; Thu, 04 Mar 2010 14:13:50 +0000 (GMT)
Received: from conversion-daemon.mail-apac.sun.com by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYR00D00GKA2T00@mail-apac.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 22:13:50 +0800 (SGT)
Received: from [125.33.137.228] ([unknown] [125.33.137.228])
 by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYR00E21GUW2SL0@mail-apac.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 22:13:50 +0800 (SGT)
Date: Thu, 04 Mar 2010 22:13:44 +0800
From: Frank Che <Frank.Che@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8FAE5E.4060001@sun.com>
Sender: Frank.Che@sun.com
To: Sebastien Roy <Sebastien.Roy@sun.com>
Cc: Bill Yan <Xue-Yang.Yan@sun.com>, Jack Schwartz <Jack.A.Schwartz@sun.com>,
        PSARC-ext@sun.com,
        zhongyuan sun - Sun Microsystems - Beijing China
 <Zhongyuan.Sun@sun.com>,
        "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>
Message-id: <4B8FC018.4070009@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8F7957.1040401@sun.com> <4B8FAE5E.4060001@sun.com>
User-Agent: Thunderbird 2.0.0.23 (Windows/20090812)
Status: RO
Content-Length: 2809

Sebastien Roy Wrote:
> Hi Bill,
>
> On 03/ 4/10 04:11 AM, Bill Yan wrote:
>>>>> 4.4 Exported Interfaces
>>>>>
>>>>> Interface | Stability | Description
>>>>> ------------------------+-------------+---------------------------------- 
>>>>>
>>>>>
>>>>> /usr/ddu/ddu-text.py | Uncommitted | Command to invoke text mode UI
>>>>
>>>>
>>>> This doesn't seem right. There's already a /usr/bin/ddu command, and
>>>> that appears to be the public interface (not the /usr/ddu/ddu.py
>>>> script to which the /usr/bin/ddu symbolic link points to). Isn't
>>>> /usr/ddu/ddu-text.py actually a Project Private interface?
>>>
>>> I'll let DDU team address this.
>>
>> ddu-text.py is not a Project Private interface, it's used by text-mode
>> installer to install missing drivers during the text-mode OS 
>> installation.
>
> You've just described a Project Private interface.  It's not used 
> directly by the user; it's merely an implementation detail of the 
> text-mode installer.
>
Enhancements to DDU (Device Driver Utility) described in this project 
material are not supposed to be used by public users. They are supposed 
to be called by the relevant Solaris installation enhancement projects. 
I have suggested the project team to use 'Consolidation Private' as the 
interface stability.
> How does the user invoke the text-mode installer?  Whatever that is 
> should be your public interface.
The text-mode installer is not in the scope of this project, it's 
another Solaris installation project that consumes this DDU interface to 
implement driver update functions.
>
>>>> A ddu(1M) man page would help.
>>>
>>> I'll let the DDU team address this.
>>
>> ddu --silent is not a public interface, it's only used to invoke the GUI
>> DDU in silent mode by GNOME autostart application. And ddu is a GUI
>> application that has an on-line help.
>
> Okay, so what I'm asking for is a description of what --silent mode 
> does.  Your materials go out of their way to state that the ddu 
> command gets called with --silent but doesn't explain what that 
> means.  Given your statement above regarding its application to the 
> GUI, what does it have to do with a text mode UI (i.e., why is that 
> relevant to this case?).
I will leave it to the project team to explain what --silent does.

This interface, --silent (should be added to the project description 
material later), has nothing to do with the text mode UI.

Actually, this project is a sum up of several enhancements to the DDU 
product.

1. Provide a text mode DDU UI, to be consumed by the Solaris text mode 
installer.
2. Provide the '--silent' interface, to be consumed by the Solaris 
LiveCD to give better user experience on driver update.
3. Provide common libraries, to be consumed by the Solaris AI 
installation enhancements and DDU UIs.

-Frank


From Alan.Coopersmith@sun.com Thu Mar  4 07:47:52 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o24FlqXM016111
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 07:47:52 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o24Flktt011407
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 4 Mar 2010 08:47:52 -0700 (MST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYR00I53L7RFA00@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 07:47:51 -0800 (PST)
Received: from sca-es-mail-1.sun.com ([192.18.43.132])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYR005OYL7QOTD0@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 07:47:50 -0800 (PST)
Received: from fe-sfbay-10.sun.com ([192.18.43.129])
	by sca-es-mail-1.sun.com (8.13.7+Sun/8.12.9) with ESMTP id o24FllQ1004008	for
 <PSARC-ext@Sun.COM>; Thu, 04 Mar 2010 07:47:50 -0800 (PST)
Received: from conversion-daemon.fe-sfbay-10.sun.com by fe-sfbay-10.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYR00100L1IOH00@fe-sfbay-10.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 07:47:47 -0800 (PST)
Received: from [129.145.155.53] ([unknown] [129.145.155.53])
 by fe-sfbay-10.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYR00LT2L7LX380@fe-sfbay-10.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 07:47:45 -0800 (PST)
Date: Thu, 04 Mar 2010 07:47:45 -0800
From: Alan Coopersmith <Alan.Coopersmith@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8F7957.1040401@sun.com>
Sender: Alan.Coopersmith@sun.com
To: Bill Yan <Xue-Yang.Yan@sun.com>
Cc: Sebastien Roy <Sebastien.Roy@sun.com>,
        Jack Schwartz <Jack.A.Schwartz@sun.com>, Frank Che <Frank.Che@sun.com>,
        PSARC-ext@sun.com,
        zhongyuan sun - Sun Microsystems - Beijing China
 <Zhongyuan.Sun@sun.com>,
        "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>
Message-id: <4B8FD621.1040500@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
X-Enigmail-Version: 0.95.1
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8F7957.1040401@sun.com>
User-Agent: Thunderbird 2.0.0.23 (X11/20090926)
Status: RO
Content-Length: 784

Bill Yan wrote:
>>> Has the project team been in contact with the pkg team to ensure that
>>> drivers installed using this tool won't be automatically removed from
>>> their installed locations during pkg image-update?
>>
>> If SVR4 packages are added and their files are orthogonal to IPS
>> packages, then IPS will leave those files alone. DDU will use the pkg
>> command to install IPS packages.

Actually, SVR4 package files will be moved to "lost+found" by IPS image-update
in some cases - this has bitten users with the SVR4 installed NVIDIA driver
on a number of occasions, but I believe that's considered a bug in IPS, not
a part of it's architectural design.

-- 
	-Alan Coopersmith-           alan.coopersmith@sun.com
	 Oracle Solaris Platform Engineering: X Window System


From Sebastien.Roy@sun.com Thu Mar  4 07:51:23 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o24FpNZv016128
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 07:51:23 -0800 (PST)
Received: from brm-avmta-1.central.sun.com (brm-avmta-1.Central.Sun.COM [129.147.4.11])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o24FpIpV014075
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 4 Mar 2010 08:51:23 -0700 (MST)
Received: from pmxchannel-daemon.brm-avmta-1.central.sun.com by
 brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYR00F3HLDM3700@brm-avmta-1.central.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 08:51:23 -0700 (MST)
Received: from brmea-mail-2.sun.com ([192.18.98.43])
 by brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYR00COYLDM3630@brm-avmta-1.central.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 08:51:22 -0700 (MST)
Received: from fe-amer-09.sun.com ([192.18.109.79])
	by brmea-mail-2.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o24FpMG3002233	for
 <PSARC-ext@Sun.COM>; Thu, 04 Mar 2010 15:51:22 +0000 (GMT)
Received: from conversion-daemon.mail-amer.sun.com by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYR00300K39XM00@mail-amer.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 08:51:22 -0700 (MST)
Received: from [129.148.174.103] ([unknown] [129.148.174.103])
 by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYR008NSLD9UYF0@mail-amer.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 08:51:10 -0700 (MST)
Date: Thu, 04 Mar 2010 10:48:26 -0500
From: Sebastien Roy <Sebastien.Roy@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8FC018.4070009@sun.com>
Sender: Sebastien.Roy@sun.com
To: Frank Che <Frank.Che@sun.com>
Cc: Bill Yan <Xue-Yang.Yan@sun.com>, Jack Schwartz <Jack.A.Schwartz@sun.com>,
        PSARC-ext@sun.com,
        zhongyuan sun - Sun Microsystems - Beijing China
 <Zhongyuan.Sun@sun.com>,
        "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>
Message-id: <4B8FD64A.4070107@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8F7957.1040401@sun.com> <4B8FAE5E.4060001@sun.com>
 <4B8FC018.4070009@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100131
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 3569

On 03/ 4/10 09:13 AM, Frank Che wrote:
> Sebastien Roy Wrote:
>> Hi Bill,
>>
>> On 03/ 4/10 04:11 AM, Bill Yan wrote:
>>>>>> 4.4 Exported Interfaces
>>>>>>
>>>>>> Interface | Stability | Description
>>>>>> ------------------------+-------------+----------------------------------
>>>>>>
>>>>>>
>>>>>> /usr/ddu/ddu-text.py | Uncommitted | Command to invoke text mode UI
>>>>>
>>>>>
>>>>> This doesn't seem right. There's already a /usr/bin/ddu command, and
>>>>> that appears to be the public interface (not the /usr/ddu/ddu.py
>>>>> script to which the /usr/bin/ddu symbolic link points to). Isn't
>>>>> /usr/ddu/ddu-text.py actually a Project Private interface?
>>>>
>>>> I'll let DDU team address this.
>>>
>>> ddu-text.py is not a Project Private interface, it's used by text-mode
>>> installer to install missing drivers during the text-mode OS
>>> installation.
>>
>> You've just described a Project Private interface. It's not used
>> directly by the user; it's merely an implementation detail of the
>> text-mode installer.
>>
> Enhancements to DDU (Device Driver Utility) described in this project
> material are not supposed to be used by public users. They are supposed
> to be called by the relevant Solaris installation enhancement projects.
> I have suggested the project team to use 'Consolidation Private' as the
> interface stability.

What is the rationale behind restricting the use of the text mode UI to 
install-time only, but letting them use the GUI at any time?  This seems 
arbitrarily confusing.

Is the python API also intended to be Consolidation Private?  That would 
make more sense to me.

>> How does the user invoke the text-mode installer? Whatever that is
>> should be your public interface.
> The text-mode installer is not in the scope of this project, it's
> another Solaris installation project that consumes this DDU interface to
> implement driver update functions.

That was a typo, I meant text mode DDU.

>>
>>>>> A ddu(1M) man page would help.
>>>>
>>>> I'll let the DDU team address this.
>>>
>>> ddu --silent is not a public interface, it's only used to invoke the GUI
>>> DDU in silent mode by GNOME autostart application. And ddu is a GUI
>>> application that has an on-line help.
>>
>> Okay, so what I'm asking for is a description of what --silent mode
>> does. Your materials go out of their way to state that the ddu command
>> gets called with --silent but doesn't explain what that means. Given
>> your statement above regarding its application to the GUI, what does
>> it have to do with a text mode UI (i.e., why is that relevant to this
>> case?).
> I will leave it to the project team to explain what --silent does.
>
> This interface, --silent (should be added to the project description
> material later), has nothing to do with the text mode UI.

I don't understand what "should be added to the project description 
material later" means.  It's in this case's spec right now, and so I 
assumed that it was in fact part of what this case is delivering right 
now (not later).

>
> Actually, this project is a sum up of several enhancements to the DDU
> product.
>
> 1. Provide a text mode DDU UI, to be consumed by the Solaris text mode
> installer.
> 2. Provide the '--silent' interface, to be consumed by the Solaris
> LiveCD to give better user experience on driver update.
> 3. Provide common libraries, to be consumed by the Solaris AI
> installation enhancements and DDU UIs.

Okay, in that case please add this to the table of exported interfaces 
and clarify its semantics in the spec.

Thanks,
-Seb

From casper@holland.sun.com Thu Mar  4 07:54:58 2010
Received: from sunmail6brm.central.sun.com (sunmail6brm.Central.Sun.COM [129.147.4.169])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o24Fsw7r016412
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 07:54:58 -0800 (PST)
Received: from brm-avmta-1.central.sun.com (brm-avmta-1.Central.Sun.COM [129.147.4.11])
	by sunmail6brm.central.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o24Fstmx002200
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@Sun.COM>; Thu, 4 Mar 2010 09:54:58 -0600 (CST)
Received: from pmxchannel-daemon.brm-avmta-1.central.sun.com by
 brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYR00F0DLJKMM00@brm-avmta-1.central.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 08:54:56 -0700 (MST)
Received: from dm-holland-01.uk.sun.com ([129.156.101.192])
 by brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYR00CD6LJJ3640@brm-avmta-1.central.sun.com> for
 PSARC-ext@Sun.COM (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 08:54:56 -0700 (MST)
Received: from holland (room101.Holland.Sun.COM [10.16.117.40])
	by dm-holland-01.uk.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4)
 with ESMTP id o24FsqmZ019233; Thu, 04 Mar 2010 15:54:52 +0000 (GMT)
Date: Thu, 04 Mar 2010 16:54:52 +0100
From: Casper.Dik@sun.com
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8FD621.1040500@sun.com>
Sender: casper@holland.sun.com
To: Alan Coopersmith <Alan.Coopersmith@sun.com>
Cc: Bill Yan <Xue-Yang.Yan@sun.com>, Sebastien Roy <Sebastien.Roy@sun.com>,
        Jack Schwartz <Jack.A.Schwartz@sun.com>, Frank Che <Frank.Che@sun.com>,
        PSARC-ext@sun.com,
        zhongyuan sun - Sun Microsystems - Beijing China
 <Zhongyuan.Sun@sun.com>,
        "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>
Message-id: <201003041554.o24FsqmZ019233@dm-holland-01.uk.sun.com>
MIME-version: 1.0
Content-type: text/plain; charset=us-ascii
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8F7957.1040401@sun.com> <4B8FD621.1040500@sun.com>
Status: RO
Content-Length: 971


>Bill Yan wrote:
>>>> Has the project team been in contact with the pkg team to ensure that
>>>> drivers installed using this tool won't be automatically removed from
>>>> their installed locations during pkg image-update?
>>>
>>> If SVR4 packages are added and their files are orthogonal to IPS
>>> packages, then IPS will leave those files alone. DDU will use the pkg
>>> command to install IPS packages.
>
>Actually, SVR4 package files will be moved to "lost+found" by IPS image-update
>in some cases - this has bitten users with the SVR4 installed NVIDIA driver
>on a number of occasions, but I believe that's considered a bug in IPS, not
>a part of it's architectural design.


Or any other files installed not by IPS.

IPS also fails when you remove a file and IPS then tries to remove the 
package, at least during update-image.  IPS should behave more like SVR4: 
if the directory is not empty, keep it.  If a file was already removed, 
don't complain.

Casper


From Sebastien.Roy@sun.com Thu Mar  4 08:13:23 2010
Received: from sunmail3mpk.sfbay.sun.com (sunmail3mpk.SFBay.Sun.COM [129.146.11.52])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o24GDNh3016758
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 08:13:23 -0800 (PST)
Received: from brm-avmta-1.central.sun.com (brm-avmta-1.Central.Sun.COM [129.147.4.11])
	by sunmail3mpk.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o24GDLZB019005
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 4 Mar 2010 08:13:23 -0800 (PST)
Received: from pmxchannel-daemon.brm-avmta-1.central.sun.com by
 brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYR00H09MEBJS00@brm-avmta-1.central.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 09:13:23 -0700 (MST)
Received: from brmea-mail-4.sun.com ([192.18.98.36])
 by brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYR00CTAMEA3960@brm-avmta-1.central.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 09:13:22 -0700 (MST)
Received: from fe-amer-09.sun.com ([192.18.109.79])
	by brmea-mail-4.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o24GDMvW007431	for
 <PSARC-ext@Sun.COM>; Thu, 04 Mar 2010 16:13:22 +0000 (GMT)
Received: from conversion-daemon.mail-amer.sun.com by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYR00J00LSDR700@mail-amer.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 09:13:22 -0700 (MST)
Received: from [129.148.174.103] ([unknown] [129.148.174.103])
 by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYR006JPME51690@mail-amer.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 09:13:18 -0700 (MST)
Date: Thu, 04 Mar 2010 11:10:34 -0500
From: Sebastien Roy <Sebastien.Roy@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8EE62F.7010905@opensolaris.org>
Sender: Sebastien.Roy@sun.com
To: Shawn Walker <swalker@opensolaris.org>
Cc: Jack Schwartz <Jack.A.Schwartz@sun.com>, PSARC-ext@sun.com,
        Bill Yan <Xue-Yang.Yan@sun.com>, Frank Che <Frank.Che@sun.com>
Message-id: <4B8FDB7A.9070404@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8EE62F.7010905@opensolaris.org>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100131
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 2715

Hi Shawn,

On 03/ 3/10 05:43 PM, Shawn Walker wrote:
> On 03/ 3/10 03:25 PM, Jack Schwartz wrote:
>> Hi Sebastien.
>>
>> While I am not part of the DDU team, I have been working closely with
>> them since my project depends heavily on their product. I worked with
>> them on forging the specification.
>>
>> I'll answer the questions I know about and leave the others.
>>
>> On 03/03/10 07:26, Sebastien Roy wrote:
>>> On 02/26/10 01:17 AM, Frank Che wrote:
>>>>> 4.2.3 Enhance driver installation capability
>>>>> Current Device Driver Utility only installs driver from IPS
>>>>> repository. In
>>>>> case the driver supplier provides driver in other format, or network
>>>>> connection is not available(for example, network controller driver is
>>>>> missing), the user need to install driver(s) in other format. In this
>>>>> project, Device Driver Utility will be enhanced to be able to install
>>>>> driver(s) in P5I [3], DU-Image [4] and SVR4 [5] formats via call "pkg"
>>>>> and
>>>>> "pkgadd" commands.
>>>
>>> Has the project team been in contact with the pkg team to ensure that
>>> drivers installed using this tool won't be automatically removed from
>>> their installed locations during pkg image-update?
>> If SVR4 packages are added and their files are orthogonal to IPS
>> packages, then IPS will leave those files alone. DDU will use the pkg
>> command to install IPS packages.
>
> That isn't quite true. If those files are installed into directories
> that belong to pkg(5) packages, they may be moved out of the way during
> uninstall, install, or update operations (salvaged to /var/pkg/lost+found).

That's what I thought.  There are two resulting questions for the 
project team:

1. We're talking specifically about /kernel/drv and applicable 
subdirectories.  If this utility is installing files in those 
directories only to have them removed when pkg image-update is run, then 
this case can't possibly go forward status-quo.  Either there's an 
architectural bug in IPS that needs to get fixed, or this case needs to 
somehow be made to work with IPS and its constraints.

2. What about this "DU-Image" format that the project team references 
above?  What is this?  If a "fix" is to have pkg play nice with SVR4 
packaged files, does that leave this format in the cold?

In general, if the OS moves drivers installed using that format, then I 
think this project has more work to do to make sure that this utility 
and ips work together.  If these are simply bugs in IPS (as Alan seems 
to be hinting at in a separate message), then I would like the project 
team to confirm this and cite appropriate bugs with confirmation from 
the IPS team that these will in fact be addressed.

-Seb

From Sebastien.Roy@sun.com Thu Mar  4 08:20:18 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o24GKIDP016821
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 08:20:18 -0800 (PST)
Received: from brm-avmta-1.central.sun.com (brm-avmta-1.Central.Sun.COM [129.147.4.11])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o24GKGEN039263
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 4 Mar 2010 09:20:17 -0700 (MST)
Received: from pmxchannel-daemon.brm-avmta-1.central.sun.com by
 brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYR00I21MPT7R00@brm-avmta-1.central.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 09:20:17 -0700 (MST)
Received: from brmea-mail-2.sun.com ([192.18.98.43])
 by brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYR00CO8MPS3770@brm-avmta-1.central.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 09:20:16 -0700 (MST)
Received: from fe-amer-09.sun.com ([192.18.109.79])
	by brmea-mail-2.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o24GKGMX015942	for
 <PSARC-ext@Sun.COM>; Thu, 04 Mar 2010 16:20:16 +0000 (GMT)
Received: from conversion-daemon.mail-amer.sun.com by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYR00J00LSDR700@mail-amer.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 09:20:16 -0700 (MST)
Received: from [129.148.174.103] ([unknown] [129.148.174.103])
 by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYR006GBMPE16C0@mail-amer.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 09:20:02 -0700 (MST)
Date: Thu, 04 Mar 2010 11:17:19 -0500
From: Sebastien Roy <Sebastien.Roy@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8ED3C7.5010601@sun.com>
Sender: Sebastien.Roy@sun.com
To: Jack Schwartz <Jack.A.Schwartz@sun.com>
Cc: Frank Che <Frank.Che@sun.com>, PSARC-ext@sun.com,
        Bill Yan <Xue-Yang.Yan@sun.com>
Message-id: <4B8FDD0F.6040603@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100131
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 2480

Hi Jack,

On 03/ 3/10 04:25 PM, Jack Schwartz wrote:
> On 03/03/10 07:26, Sebastien Roy wrote:
>> On 02/26/10 01:17 AM, Frank Che wrote:
>>> ------------------------+-------------+----------------------------------
>>>
>>> ddu_devscan() | Uncommitted | DDU library API [2]
>>> ------------------------+-------------+----------------------------------
>>>
>>> ddu_package_lookup() | Uncommitted | DDU library API [2]
>>> ddu_build_repo_list() | |
>>> ddu_install_package() | |
>>> ------------------------+-------------+----------------------------------
>>>
>>> ddu_repo_object | Uncommitted | DDU library object [2]
>>> ddu_package_object | |
>>> ddu_dev_data | |
>>> ------------------------+-------------+----------------------------------
>>>
>>
>> What about the library itself? Is it a C library, a python module,
>> something else...?
> The library interfaces are implemented in python. The interface modules
> call other python and shell scripts, and C programs, all delivered as
> part of the library.

Python objects and functions don't magically appear in a program's 
namespace, right?  What does a program have to do to import these 
things?  It would seem that whatever that is in the set of exported 
interfaces.

>>
>> Based on the materials, I don't understand the semantics of
>> /usr/bin/ddu --silent. Basically, I don't grok section 2.1 of your
>> "programming interface spec" (and it seems to be more than a
>> programming interface spec). You explain when --silent is used, but
>> not what it does. This seems central to the proposal, and I'd like to
>> understand this.
> The programming interface spec was written to complement the functional
> interface spec for the Driver Update project. The latter shows the
> higher-level functionality and addresses what silent mode is. I suggest
> the functional interface spec for Driver Update be moved into the case
> materials as well, for added context.
>
> For now, it is here:
> http://hub.opensolaris.org/bin/download/Project+caiman/Driver_Update/du%2Dfunc%2Dspec.txt

Yes, this would help.

>
> The GUI DDU is invoked in silent mode on the live CD, to check for
> missing drivers and install them in the background. If additional
> drivers are still missing, the DDU pops up a window to tell the user,
> and to offer to display the GUI for manual action.

This makes sense, thanks for the explanation.  Why is there not an 
equivalent check done in the background in the text-based installer?

Thanks,
-Seb

From swalker@opensolaris.org Thu Mar  4 08:29:09 2010
Received: from sunmail3mpk.sfbay.sun.com (sunmail3mpk.SFBay.Sun.COM [129.146.11.52])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o24GT9eu016877
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 08:29:09 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by sunmail3mpk.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o24GT2Fo028638;
	Thu, 4 Mar 2010 08:29:06 -0800 (PST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYR00401N3VED00@nwk-avmta-1.sfbay.Sun.COM>; Thu,
 04 Mar 2010 08:28:43 -0800 (PST)
Received: from jurassic.Eng.Sun.COM ([129.146.17.63])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYR00KNZN3VD120@nwk-avmta-1.sfbay.Sun.COM>; Thu,
 04 Mar 2010 08:28:43 -0800 (PST)
Received: from [10.7.250.123]
 (punchin-client-10-7-250-123.SFBay.Sun.COM [10.7.250.123])
	by jurassic.Eng.Sun.COM (8.14.4+Sun/8.14.4) with ESMTP id o24GSgc6948008; Thu,
 04 Mar 2010 08:28:42 -0800 (PST)
Date: Thu, 04 Mar 2010 10:28:41 -0600
From: Shawn Walker <swalker@opensolaris.org>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8FDB7A.9070404@sun.com>
To: Sebastien Roy <Sebastien.Roy@sun.com>
Cc: Jack Schwartz <Jack.A.Schwartz@sun.com>, PSARC-ext@sun.com,
        Bill Yan <Xue-Yang.Yan@sun.com>, Frank Che <Frank.Che@sun.com>,
        Danek Duvall <Danek.Duvall@sun.com>,
        Bart Smaalders <Bart.Smaalders@sun.com>
Message-id: <4B8FDFB9.1040504@opensolaris.org>
MIME-version: 1.0
Content-type: text/plain; charset=UTF-8; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8EE62F.7010905@opensolaris.org> <4B8FDB7A.9070404@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100131
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 2170

On 03/ 4/10 10:10 AM, Sebastien Roy wrote:
> On 03/ 3/10 05:43 PM, Shawn Walker wrote:
>> On 03/ 3/10 03:25 PM, Jack Schwartz wrote:
...
>>> If SVR4 packages are added and their files are orthogonal to IPS
>>> packages, then IPS will leave those files alone. DDU will use the pkg
>>> command to install IPS packages.
>>
>> That isn't quite true. If those files are installed into directories
>> that belong to pkg(5) packages, they may be moved out of the way during
>> uninstall, install, or update operations (salvaged to
>> /var/pkg/lost+found).
>
> That's what I thought. There are two resulting questions for the project
> team:
>
> 1. We're talking specifically about /kernel/drv and applicable
> subdirectories. If this utility is installing files in those directories
> only to have them removed when pkg image-update is run, then this case
> can't possibly go forward status-quo. Either there's an architectural
> bug in IPS that needs to get fixed, or this case needs to somehow be
> made to work with IPS and its constraints.

There are certain bits of integration functionality that have not yet 
been implemented for SVR4 packages.  However, I personally don't know 
how deep the integration is planned to be.

> 2. What about this "DU-Image" format that the project team references
> above? What is this? If a "fix" is to have pkg play nice with SVR4
> packaged files, does that leave this format in the cold?
>
> In general, if the OS moves drivers installed using that format, then I
> think this project has more work to do to make sure that this utility
> and ips work together. If these are simply bugs in IPS (as Alan seems to
> be hinting at in a separate message), then I would like the project team
> to confirm this and cite appropriate bugs with confirmation from the IPS
> team that these will in fact be addressed.

There is an ongoing discussion regarding unpackaged bits and their 
storage and how to deal with them.  I'd suggest that the project team 
send an inquiry to pkg-discuss detailing their plans so that the pkg(5) 
team can help determine what the short-term and long-term plans should 
be in this area.

Cheers,
-- 
Shawn Walker

From Jack.A.Schwartz@sun.com Thu Mar  4 16:38:09 2010
Received: from sunmail3mpk.sfbay.sun.com (sunmail3mpk.SFBay.Sun.COM [129.146.11.52])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o250c9oU023603
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 16:38:09 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by sunmail3mpk.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o250c88T024919
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 4 Mar 2010 16:38:09 -0800 (PST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYS00C0Z9RLMV00@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 16:38:09 -0800 (PST)
Received: from sca-es-mail-2.sun.com ([192.18.43.133])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYS00B9D9RJN000@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 16:38:08 -0800 (PST)
Received: from fe-sfbay-10.sun.com ([192.18.43.129])
	by sca-es-mail-2.sun.com (8.13.7+Sun/8.12.9) with ESMTP id o250c7oN010588	for
 <PSARC-ext@Sun.COM>; Thu, 04 Mar 2010 16:38:07 -0800 (PST)
Received: from conversion-daemon.fe-sfbay-10.sun.com by fe-sfbay-10.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYS00M009R8TI00@fe-sfbay-10.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 16:38:07 -0800 (PST)
Received: from [129.146.226.171] ([unknown] [129.146.226.171])
 by fe-sfbay-10.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYS0020S9R7QRE0@fe-sfbay-10.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 16:38:07 -0800 (PST)
Date: Thu, 04 Mar 2010 16:36:55 -0800
From: Jack Schwartz <Jack.A.Schwartz@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8FD64A.4070107@sun.com>
Sender: Jack.A.Schwartz@sun.com
To: Sebastien Roy <Sebastien.Roy@sun.com>
Cc: Frank Che <Frank.Che@sun.com>, Bill Yan <Xue-Yang.Yan@sun.com>,
        PSARC-ext@sun.com,
        zhongyuan sun - Sun Microsystems - Beijing China
 <Zhongyuan.Sun@sun.com>,
        "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>
Message-id: <4B905227.9050602@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8F7957.1040401@sun.com> <4B8FAE5E.4060001@sun.com>
 <4B8FC018.4070009@sun.com> <4B8FD64A.4070107@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.5) Gecko/20100103
 Lightning/1.0b1 Thunderbird/3.0
Status: RO
Content-Length: 4878

Hi Sebastien.

(Reply 1 of 3)

On 03/04/10 07:48, Sebastien Roy wrote:
> On 03/ 4/10 09:13 AM, Frank Che wrote:
>> Sebastien Roy Wrote:
>>> Hi Bill,
>>>
>>> On 03/ 4/10 04:11 AM, Bill Yan wrote:
>>>>>>> 4.4 Exported Interfaces
>>>>>>>
>>>>>>> Interface | Stability | Description
>>>>>>> ------------------------+-------------+---------------------------------- 
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> /usr/ddu/ddu-text.py | Uncommitted | Command to invoke text mode UI
>>>>>>
>>>>>>
>>>>>> This doesn't seem right. There's already a /usr/bin/ddu command, and
>>>>>> that appears to be the public interface (not the /usr/ddu/ddu.py
>>>>>> script to which the /usr/bin/ddu symbolic link points to). Isn't
>>>>>> /usr/ddu/ddu-text.py actually a Project Private interface?
>>>>>
>>>>> I'll let DDU team address this.
>>>>
>>>> ddu-text.py is not a Project Private interface, it's used by text-mode
>>>> installer to install missing drivers during the text-mode OS
>>>> installation.
>>>
>>> You've just described a Project Private interface. It's not used
>>> directly by the user; it's merely an implementation detail of the
>>> text-mode installer.
>>>
>> Enhancements to DDU (Device Driver Utility) described in this project
>> material are not supposed to be used by public users. They are supposed
>> to be called by the relevant Solaris installation enhancement projects.
>> I have suggested the project team to use 'Consolidation Private' as the
>> interface stability.
>
> What is the rationale behind restricting the use of the text mode UI 
> to install-time only, but letting them use the GUI at any time?  This 
> seems arbitrarily confusing.
The GUI has been available after install time, and will remain so.  The 
text mode UI will be copied onto the installed target the same way.  
There won't be as obvious a way of finding the text mode UI like the 
GUI, just because the environment is not a windows-based environment 
with start menus, popups and icons.

The importance of this ARC case's objects is at installation time, 
however.  They make the DDU available to install missing drivers even 
before the installer runs, so that the installer can run on a system 
with a full complement of devices.  An installed device can include, 
potentially, the target disk to install on.

Both the GUI and text-mode UI are separate programs from their 
environment's respective installers.
>
> Is the python API also intended to be Consolidation Private?  That 
> would make more sense to me.
>
>>> How does the user invoke the text-mode installer? Whatever that is
>>> should be your public interface.
>> The text-mode installer is not in the scope of this project, it's
>> another Solaris installation project that consumes this DDU interface to
>> implement driver update functions.
>
> That was a typo, I meant text mode DDU.
>
>>>
>>>>>> A ddu(1M) man page would help.
>>>>>
>>>>> I'll let the DDU team address this.
>>>>
>>>> ddu --silent is not a public interface, it's only used to invoke 
>>>> the GUI
>>>> DDU in silent mode by GNOME autostart application. And ddu is a GUI
>>>> application that has an on-line help.
>>>
>>> Okay, so what I'm asking for is a description of what --silent mode
>>> does. Your materials go out of their way to state that the ddu command
>>> gets called with --silent but doesn't explain what that means. Given
>>> your statement above regarding its application to the GUI, what does
>>> it have to do with a text mode UI (i.e., why is that relevant to this
>>> case?).
>> I will leave it to the project team to explain what --silent does.
>>
>> This interface, --silent (should be added to the project description
>> material later), has nothing to do with the text mode UI.
>
> I don't understand what "should be added to the project description 
> material later" means.  It's in this case's spec right now, and so I 
> assumed that it was in fact part of what this case is delivering right 
> now (not later).
>
>>
>> Actually, this project is a sum up of several enhancements to the DDU
>> product.
>>
>> 1. Provide a text mode DDU UI, to be consumed by the Solaris text mode
>> installer.
Being separate programs, and that the UI will be delivered to the target 
disk like most other programs in the booted install environment, saying 
"consumed by the installer" is incorrect.  How about saying something like:

"Provide a text mode DDU UI, to help furnish a more device-complete 
installation environment for the text-mode installer."
>> 2. Provide the '--silent' interface, to be consumed by the Solaris
>> LiveCD to give better user experience on driver update.
>> 3. Provide common libraries, to be consumed by the Solaris AI
>> installation enhancements and DDU UIs.
>
> Okay, in that case please add this to the table of exported interfaces 
> and clarify its semantics in the spec.
     Thanks,
     Jack
>
> Thanks,
> -Seb


From Jack.A.Schwartz@Sun.COM Thu Mar  4 16:39:40 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o250ddIW023615
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 16:39:40 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o250ddsi021988
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 4 Mar 2010 17:39:39 -0700 (MST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYS00D019U31V00@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 16:39:39 -0800 (PST)
Received: from sca-es-mail-1.sun.com ([192.18.43.132])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYS00BDC9U3N500@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 16:39:39 -0800 (PST)
Received: from fe-sfbay-09.sun.com ([192.18.43.129])
	by sca-es-mail-1.sun.com (8.13.7+Sun/8.12.9) with ESMTP id o250dcmK000472	for
 <PSARC-ext@Sun.COM>; Thu, 04 Mar 2010 16:39:38 -0800 (PST)
Received: from conversion-daemon.fe-sfbay-09.sun.com by fe-sfbay-09.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYS00B0099ASE00@fe-sfbay-09.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 16:39:38 -0800 (PST)
Received: from [129.146.226.171] ([unknown] [129.146.226.171])
 by fe-sfbay-09.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYS003V29TYOA00@fe-sfbay-09.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 16:39:38 -0800 (PST)
Date: Thu, 04 Mar 2010 16:38:33 -0800
From: Jack Schwartz <Jack.A.Schwartz@Sun.COM>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8FDB7A.9070404@sun.com>
Sender: Jack.A.Schwartz@Sun.COM
To: Sebastien Roy <Sebastien.Roy@Sun.COM>
Cc: Shawn Walker <swalker@opensolaris.org>, PSARC-ext@Sun.COM,
        Bill Yan <Xue-Yang.Yan@Sun.COM>, Frank Che <Frank.Che@Sun.COM>
Message-id: <4B905289.8000905@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8EE62F.7010905@opensolaris.org> <4B8FDB7A.9070404@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.5) Gecko/20100103
 Lightning/1.0b1 Thunderbird/3.0
Status: RO
Content-Length: 3172

Hi Sebastien.

(Reply 2 of 3)

On 03/04/10 08:10, Sebastien Roy wrote:
> Hi Shawn,
>
> On 03/ 3/10 05:43 PM, Shawn Walker wrote:
>> On 03/ 3/10 03:25 PM, Jack Schwartz wrote:
>>> Hi Sebastien.
>>>
>>> While I am not part of the DDU team, I have been working closely with
>>> them since my project depends heavily on their product. I worked with
>>> them on forging the specification.
>>>
>>> I'll answer the questions I know about and leave the others.
>>>
>>> On 03/03/10 07:26, Sebastien Roy wrote:
>>>> On 02/26/10 01:17 AM, Frank Che wrote:
>>>>>> 4.2.3 Enhance driver installation capability
>>>>>> Current Device Driver Utility only installs driver from IPS
>>>>>> repository. In
>>>>>> case the driver supplier provides driver in other format, or network
>>>>>> connection is not available(for example, network controller 
>>>>>> driver is
>>>>>> missing), the user need to install driver(s) in other format. In 
>>>>>> this
>>>>>> project, Device Driver Utility will be enhanced to be able to 
>>>>>> install
>>>>>> driver(s) in P5I [3], DU-Image [4] and SVR4 [5] formats via call 
>>>>>> "pkg"
>>>>>> and
>>>>>> "pkgadd" commands.
>>>>
>>>> Has the project team been in contact with the pkg team to ensure that
>>>> drivers installed using this tool won't be automatically removed from
>>>> their installed locations during pkg image-update?
>>> If SVR4 packages are added and their files are orthogonal to IPS
>>> packages, then IPS will leave those files alone. DDU will use the pkg
>>> command to install IPS packages.
>>
>> That isn't quite true. If those files are installed into directories
>> that belong to pkg(5) packages, they may be moved out of the way during
>> uninstall, install, or update operations (salvaged to 
>> /var/pkg/lost+found).
>
> That's what I thought.  There are two resulting questions for the 
> project team:
>
> 1. We're talking specifically about /kernel/drv and applicable 
> subdirectories.  If this utility is installing files in those 
> directories only to have them removed when pkg image-update is run, 
> then this case can't possibly go forward status-quo.  Either there's 
> an architectural bug in IPS that needs to get fixed, or this case 
> needs to somehow be made to work with IPS and its constraints.
Yes, else a customer's system can break down the road when they try an 
image-update.
>
> 2. What about this "DU-Image" format that the project team references 
> above?  What is this?  If a "fix" is to have pkg play nice with SVR4 
> packaged files, does that leave this format in the cold?
DU-image format is the ITU format of Solaris 10.  This is basically a 
datastream SVR4 package in a wrapper, so it has the same issues as SVR4 
packages do.
>
> In general, if the OS moves drivers installed using that format, then 
> I think this project has more work to do to make sure that this 
> utility and ips work together.  If these are simply bugs in IPS (as 
> Alan seems to be hinting at in a separate message), then I would like 
> the project team to confirm this and cite appropriate bugs with 
> confirmation from the IPS team that these will in fact be addressed.
OK.

     Thanks,
     Jack
>
> -Seb


From Frank.Che@sun.com Thu Mar  4 17:30:25 2010
Received: from sunmail6brm.central.sun.com (sunmail6brm.Central.Sun.COM [129.147.4.169])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o251UP7S024374
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 17:30:25 -0800 (PST)
Received: from brm-avmta-1.central.sun.com (brm-avmta-1.Central.Sun.COM [129.147.4.11])
	by sunmail6brm.central.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o251UO7F003947
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 4 Mar 2010 19:30:24 -0600 (CST)
Received: from pmxchannel-daemon.brm-avmta-1.central.sun.com by
 brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYS0060PC6O9M00@brm-avmta-1.central.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 18:30:24 -0700 (MST)
Received: from sineb-mail-2.sun.com ([192.18.19.7])
 by brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYS005K2C6LAK00@brm-avmta-1.central.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 18:30:22 -0700 (MST)
Received: from fe-apac-05.sun.com
 (fe-apac-05.sun.com [192.18.19.176] (may be forged))
	by sineb-mail-2.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o251ULUe006740	for
 <PSARC-ext@Sun.COM>; Fri, 05 Mar 2010 01:30:21 +0000 (GMT)
Received: from conversion-daemon.mail-apac.sun.com by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYS00500BTDOE00@mail-apac.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Fri, 05 Mar 2010 09:30:21 +0800 (SGT)
Received: from [129.158.218.60] ([unknown] [129.158.218.60])
 by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYS00E8DC6GOA50@mail-apac.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Fri, 05 Mar 2010 09:30:21 +0800 (SGT)
Date: Fri, 05 Mar 2010 09:30:09 +0800
From: Frank Che <Frank.Che@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8FD64A.4070107@sun.com>
Sender: Frank.Che@sun.com
To: Sebastien Roy <Sebastien.Roy@sun.com>
Cc: Bill Yan <Xue-Yang.Yan@sun.com>, Jack Schwartz <Jack.A.Schwartz@sun.com>,
        PSARC-ext@sun.com,
        zhongyuan sun - Sun Microsystems - Beijing China
 <Zhongyuan.Sun@sun.com>,
        "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>
Message-id: <4B905EA1.3040802@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8F7957.1040401@sun.com> <4B8FAE5E.4060001@sun.com>
 <4B8FC018.4070009@sun.com> <4B8FD64A.4070107@sun.com>
User-Agent: Thunderbird 2.0.0.19 (X11/20090218)
Status: RO
Content-Length: 950

Sebastien Roy wrote:
> I don't understand what "should be added to the project description 
> material later" means.  It's in this case's spec right now, and so I 
> assumed that it was in fact part of what this case is delivering right 
> now (not later).
I just mean this interface is currently not in the exported interface 
list, should be added later when updating the case material.

-Frank
>
>>
>> Actually, this project is a sum up of several enhancements to the DDU
>> product.
>>
>> 1. Provide a text mode DDU UI, to be consumed by the Solaris text mode
>> installer.
>> 2. Provide the '--silent' interface, to be consumed by the Solaris
>> LiveCD to give better user experience on driver update.
>> 3. Provide common libraries, to be consumed by the Solaris AI
>> installation enhancements and DDU UIs.
>
> Okay, in that case please add this to the table of exported interfaces 
> and clarify its semantics in the spec.
>
> Thanks,
> -Seb

From Jack.A.Schwartz@sun.com Thu Mar  4 17:31:48 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o251VmdA024391
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 17:31:48 -0800 (PST)
Received: from brm-avmta-1.central.sun.com (brm-avmta-1.Central.Sun.COM [129.147.4.11])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o251VlYG056743
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 4 Mar 2010 18:31:48 -0700 (MST)
Received: from pmxchannel-daemon.brm-avmta-1.central.sun.com by
 brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYS0060DC8ZE600@brm-avmta-1.central.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 18:31:47 -0700 (MST)
Received: from sca-es-mail-1.sun.com ([192.18.43.132])
 by brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYS005O9C8ZAF00@brm-avmta-1.central.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 18:31:47 -0700 (MST)
Received: from fe-sfbay-10.sun.com ([192.18.43.129])
	by sca-es-mail-1.sun.com (8.13.7+Sun/8.12.9) with ESMTP id o251Vl8w004257	for
 <PSARC-ext@Sun.COM>; Thu, 04 Mar 2010 17:31:47 -0800 (PST)
Received: from conversion-daemon.fe-sfbay-10.sun.com by fe-sfbay-10.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYS00E00C6GY700@fe-sfbay-10.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 17:31:47 -0800 (PST)
Received: from [129.146.226.171] ([unknown] [129.146.226.171])
 by fe-sfbay-10.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYS004JOC8YU210@fe-sfbay-10.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 17:31:46 -0800 (PST)
Date: Thu, 04 Mar 2010 17:30:45 -0800
From: Jack Schwartz <Jack.A.Schwartz@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8FDD0F.6040603@sun.com>
Sender: Jack.A.Schwartz@sun.com
To: Sebastien Roy <Sebastien.Roy@sun.com>
Cc: Frank Che <Frank.Che@sun.com>, PSARC-ext@sun.com,
        Bill Yan <Xue-Yang.Yan@sun.com>
Message-id: <4B905EC5.5080604@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.5) Gecko/20100103
 Lightning/1.0b1 Thunderbird/3.0
Status: RO
Content-Length: 3317

Hi again, Sebastien.

(reply 3 of 3)

On 03/04/10 08:17, Sebastien Roy wrote:
> Hi Jack,
>
> On 03/ 3/10 04:25 PM, Jack Schwartz wrote:
>> On 03/03/10 07:26, Sebastien Roy wrote:
>>> On 02/26/10 01:17 AM, Frank Che wrote:
>>>> ------------------------+-------------+---------------------------------- 
>>>>
>>>>
>>>> ddu_devscan() | Uncommitted | DDU library API [2]
>>>> ------------------------+-------------+---------------------------------- 
>>>>
>>>>
>>>> ddu_package_lookup() | Uncommitted | DDU library API [2]
>>>> ddu_build_repo_list() | |
>>>> ddu_install_package() | |
>>>> ------------------------+-------------+---------------------------------- 
>>>>
>>>>
>>>> ddu_repo_object | Uncommitted | DDU library object [2]
>>>> ddu_package_object | |
>>>> ddu_dev_data | |
>>>> ------------------------+-------------+---------------------------------- 
>>>>
>>>>
>>>
>>> What about the library itself? Is it a C library, a python module,
>>> something else...?
>> The library interfaces are implemented in python. The interface modules
>> call other python and shell scripts, and C programs, all delivered as
>> part of the library.
>
> Python objects and functions don't magically appear in a program's 
> namespace, right?  What does a program have to do to import these 
> things?  It would seem that whatever that is in the set of exported 
> interfaces.
Being python, one imports the modules, or else imports functions and 
classes from a module. For example:

from DDU.ddu_function import ddu_build_repo_list

So make the python modules themselves part of the interface?  This makes 
sense, as if these change, the programs which reference them will break.
>>> Based on the materials, I don't understand the semantics of
>>> /usr/bin/ddu --silent. Basically, I don't grok section 2.1 of your
>>> "programming interface spec" (and it seems to be more than a
>>> programming interface spec). You explain when --silent is used, but
>>> not what it does. This seems central to the proposal, and I'd like to
>>> understand this.
>> The programming interface spec was written to complement the functional
>> interface spec for the Driver Update project. The latter shows the
>> higher-level functionality and addresses what silent mode is. I suggest
>> the functional interface spec for Driver Update be moved into the case
>> materials as well, for added context.
>>
>> For now, it is here:
>> http://hub.opensolaris.org/bin/download/Project+caiman/Driver_Update/du%2Dfunc%2Dspec.txt 
>>
>
> Yes, this would help.
>
>>
>> The GUI DDU is invoked in silent mode on the live CD, to check for
>> missing drivers and install them in the background. If additional
>> drivers are still missing, the DDU pops up a window to tell the user,
>> and to offer to display the GUI for manual action.
>
> This makes sense, thanks for the explanation.  Why is there not an 
> equivalent check done in the background in the text-based installer?
The lack of a non-windows environment to cleanly pop up a notification 
window is why the text-installer doesn't have a silent mode.  Instead, 
the text-mode UI will be invoked by the start menu;  similar to the 
start menu on S10, there is a menu item for running the installer, and 
there is another for running the text-mode DDU UI.

     Thanks,
     Jack
>
> Thanks,
> -Seb


From Frank.Che@Sun.COM Thu Mar  4 17:33:05 2010
Received: from sunmail3mpk.sfbay.sun.com (sunmail3mpk.SFBay.Sun.COM [129.146.11.52])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o251X51S024424
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 17:33:05 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by sunmail3mpk.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o251X46K021372
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Thu, 4 Mar 2010 17:33:04 -0800 (PST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYS00505CB44O00@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 17:33:04 -0800 (PST)
Received: from sineb-mail-2.sun.com ([192.18.19.7])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYS00BIQCB2N430@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 17:33:03 -0800 (PST)
Received: from fe-apac-05.sun.com
 (fe-apac-05.sun.com [192.18.19.176] (may be forged))
	by sineb-mail-2.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o251X167006807	for
 <PSARC-ext@Sun.COM>; Fri, 05 Mar 2010 01:33:01 +0000 (GMT)
Received: from conversion-daemon.mail-apac.sun.com by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYS00500BTDOE00@mail-apac.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Fri, 05 Mar 2010 09:33:01 +0800 (SGT)
Received: from [129.158.218.60] ([unknown] [129.158.218.60])
 by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYS00EAWCB0OA50@mail-apac.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Fri, 05 Mar 2010 09:33:01 +0800 (SGT)
Date: Fri, 05 Mar 2010 09:32:53 +0800
From: Frank Che <Frank.Che@Sun.COM>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8FD64A.4070107@sun.com>
Sender: Frank.Che@Sun.COM
To: PSARC-ext@Sun.COM
Cc: Sebastien Roy <Sebastien.Roy@Sun.COM>, Bill Yan <Xue-Yang.Yan@Sun.COM>,
        Jack Schwartz <Jack.A.Schwartz@Sun.COM>,
        zhongyuan sun - Sun Microsystems - Beijing China <Zhongyuan.Sun@Sun.COM>,
        "Tony.Hu@Sun.COM" <Tony.Hu@Sun.COM>
Message-id: <4B905F45.4020209@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8F7957.1040401@sun.com> <4B8FAE5E.4060001@sun.com>
 <4B8FC018.4070009@sun.com> <4B8FD64A.4070107@sun.com>
User-Agent: Thunderbird 2.0.0.19 (X11/20090218)
Status: RO
Content-Length: 89

Since the review is still on going, I'd like to extend the timer to 
03/10/2010.

-Frank

From Xue-Yang.Yan@Sun.COM Thu Mar  4 23:03:07 2010
Received: from sunmail6brm.central.sun.com (sunmail6brm.Central.Sun.COM [129.147.4.169])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o25737ao028891
	for <psarc-ext@sac.sfbay.sun.com>; Thu, 4 Mar 2010 23:03:07 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by sunmail6brm.central.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o25731oA025729
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Fri, 5 Mar 2010 01:03:06 -0600 (CST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYS0050FRL5I000@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Thu, 04 Mar 2010 23:03:05 -0800 (PST)
Received: from sineb-mail-1.sun.com ([192.18.19.6])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYS0033VRL4DA30@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Thu,
 04 Mar 2010 23:03:05 -0800 (PST)
Received: from fe-apac-06.sun.com
 (fe-apac-06.sun.com [192.18.19.177] (may be forged))
	by sineb-mail-1.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o25734rs011804	for
 <PSARC-ext@Sun.COM>; Fri, 05 Mar 2010 07:03:04 +0000 (GMT)
Received: from conversion-daemon.mail-apac.sun.com by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYS00C00RHDJ200@mail-apac.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Fri, 05 Mar 2010 15:03:04 +0800 (SGT)
Received: from [129.158.218.53] ([unknown] [129.158.218.53])
 by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYS00KB8RL24RF0@mail-apac.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Fri, 05 Mar 2010 15:03:03 +0800 (SGT)
Date: Fri, 05 Mar 2010 15:00:13 +0800
From: Bill Yan <Xue-Yang.Yan@Sun.COM>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B8FDD0F.6040603@sun.com>
Sender: Xue-Yang.Yan@Sun.COM
To: PSARC-ext@Sun.COM
Cc: Sebastien Roy <Sebastien.Roy@Sun.COM>,
        Jack Schwartz <Jack.A.Schwartz@Sun.COM>, Frank Che <Frank.Che@Sun.COM>,
        "Tony.Hu@Sun.COM" <Tony.Hu@Sun.COM>,
        zhongyuan sun - Sun Microsystems - Beijing China <Zhongyuan.Sun@Sun.COM>
Message-id: <4B90ABFD.60105@sun.com>
MIME-version: 1.0
Content-type: text/plain; format=flowed; charset=UTF-8
Content-transfer-encoding: 8BIT
X-Accept-Language: zh-cn
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; zh-CN; rv:1.7) Gecko/20060627
Status: RO
Content-Length: 4196

Hi,

Based on the feedback, we are going to make the following changes to the 
export interface:

    Interface               |  Stability              | Description
    ------------------------+-------------------------+----------------------------------
    /usr/bin/ddu-text       | Uncommitted             | Command to invoke text mode UI
    ------------------------+-------------------------+----------------------------------
    /usr/bin/ddu --silent   | Uncommitted             | Command to invoke GUI ddu in silent mode
    ------------------------+-------------------------+-------------------------------
    ddu_devscan()           | Consolidation private   | DDU library API [2]
    ------------------------+-------------+----------------------------------
    ddu_package_lookup()    | Consolidation private   | DDU library API [2]
    ddu_build_repo_list()   |                         |
    ddu_install_package()   |                         |
    ------------------------+-------------------------+----------------------------------
    ddu_repo_object         | Consolidation private   | DDU library object [2]
    ddu_package_object      |                         |
    ddu_dev_data            |                         |
    ------------------------+-------------------------+----------------------------------



And we are going to add man pages for /usr/bin/ddu-text and /usr/bin/ddu

Regarding the SVR4 and IPS package issue, Jack will help to address the 
issue and cite appropriate bugs to IPS team. Thanks a lot for Jack's help.

Regards,
Bill
Sebastien Roy 已写入:

> Hi Jack,
>
> On 03/ 3/10 04:25 PM, Jack Schwartz wrote:
>
>> On 03/03/10 07:26, Sebastien Roy wrote:
>>
>>> On 02/26/10 01:17 AM, Frank Che wrote:
>>>
>>>> ------------------------+-------------+---------------------------------- 
>>>>
>>>>
>>>> ddu_devscan() | Uncommitted | DDU library API [2]
>>>> ------------------------+-------------+---------------------------------- 
>>>>
>>>>
>>>> ddu_package_lookup() | Uncommitted | DDU library API [2]
>>>> ddu_build_repo_list() | |
>>>> ddu_install_package() | |
>>>> ------------------------+-------------+---------------------------------- 
>>>>
>>>>
>>>> ddu_repo_object | Uncommitted | DDU library object [2]
>>>> ddu_package_object | |
>>>> ddu_dev_data | |
>>>> ------------------------+-------------+---------------------------------- 
>>>>
>>>>
>>>
>>> What about the library itself? Is it a C library, a python module,
>>> something else...?
>>
>> The library interfaces are implemented in python. The interface modules
>> call other python and shell scripts, and C programs, all delivered as
>> part of the library.
>
>
> Python objects and functions don't magically appear in a program's 
> namespace, right? What does a program have to do to import these 
> things? It would seem that whatever that is in the set of exported 
> interfaces.
>
>>>
>>> Based on the materials, I don't understand the semantics of
>>> /usr/bin/ddu --silent. Basically, I don't grok section 2.1 of your
>>> "programming interface spec" (and it seems to be more than a
>>> programming interface spec). You explain when --silent is used, but
>>> not what it does. This seems central to the proposal, and I'd like to
>>> understand this.
>>
>> The programming interface spec was written to complement the functional
>> interface spec for the Driver Update project. The latter shows the
>> higher-level functionality and addresses what silent mode is. I suggest
>> the functional interface spec for Driver Update be moved into the case
>> materials as well, for added context.
>>
>> For now, it is here:
>> http://hub.opensolaris.org/bin/download/Project+caiman/Driver_Update/du%2Dfunc%2Dspec.txt 
>>
>
>
> Yes, this would help.
>
>>
>> The GUI DDU is invoked in silent mode on the live CD, to check for
>> missing drivers and install them in the background. If additional
>> drivers are still missing, the DDU pops up a window to tell the user,
>> and to offer to display the GUI for manual action.
>
>
> This makes sense, thanks for the explanation. Why is there not an 
> equivalent check done in the background in the text-based installer?
>
> Thanks,
> -Seb



From Sebastien.Roy@Sun.COM Fri Mar  5 05:21:49 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o25DLnLB019671
	for <psarc-ext@sac.sfbay.sun.com>; Fri, 5 Mar 2010 05:21:49 -0800 (PST)
Received: from nwk-avmta-2.sfbay.sun.com (nwk-avmta-2.SFBay.Sun.COM [129.145.155.6])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o25DLl84003686
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Fri, 5 Mar 2010 06:21:49 -0700 (MST)
Received: from pmxchannel-daemon.nwk-avmta-2.sfbay.sun.com by
 nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYT00D0V94CDJ00@nwk-avmta-2.sfbay.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Fri, 05 Mar 2010 05:21:48 -0800 (PST)
Received: from brmea-mail-4.sun.com ([192.18.98.36])
 by nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYT00MGM94C4YB0@nwk-avmta-2.sfbay.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Fri,
 05 Mar 2010 05:21:48 -0800 (PST)
Received: from fe-amer-09.sun.com ([192.18.109.79])
	by brmea-mail-4.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o25DLljr011944	for
 <PSARC-ext@Sun.COM>; Fri, 05 Mar 2010 13:21:47 +0000 (GMT)
Received: from conversion-daemon.mail-amer.sun.com by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYT00E008ZDWO00@mail-amer.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Fri, 05 Mar 2010 06:21:47 -0700 (MST)
Received: from [192.168.1.3] ([unknown] [173.76.16.34])
 by mail-amer.sun.com (Sun Java(tm) System Messaging Server 7u2-7.04 64bit
 (built Jul  2 2009)) with ESMTPSA id <0KYT0069R94AYBA0@mail-amer.sun.com> for
 PSARC-ext@Sun.COM (ORCPT PSARC-ext@Sun.COM); Fri,
 05 Mar 2010 06:21:47 -0700 (MST)
Date: Fri, 05 Mar 2010 08:21:46 -0500
From: Sebastien Roy <Sebastien.Roy@Sun.COM>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B905EC5.5080604@sun.com>
Sender: Sebastien.Roy@Sun.COM
To: Jack Schwartz <Jack.A.Schwartz@Sun.COM>
Cc: Frank Che <Frank.Che@Sun.COM>, PSARC-ext@Sun.COM,
        Bill Yan <Xue-Yang.Yan@Sun.COM>
Message-id: <4B91056A.4050106@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B905EC5.5080604@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100131
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 1746

Hi Jack,

On 03/ 4/10 08:30 PM, Jack Schwartz wrote:
>>> The library interfaces are implemented in python. The interface modules
>>> call other python and shell scripts, and C programs, all delivered as
>>> part of the library.
>>
>> Python objects and functions don't magically appear in a program's
>> namespace, right? What does a program have to do to import these
>> things? It would seem that whatever that is in the set of exported
>> interfaces.
> Being python, one imports the modules, or else imports functions and
> classes from a module. For example:
>
> from DDU.ddu_function import ddu_build_repo_list
>
> So make the python modules themselves part of the interface? This makes
> sense, as if these change, the programs which reference them will break.

Yes, precisely.  It may also be useful for informational purposes to 
state in the interface table where in the filesystem the module file 
will be installed (in the comments section of the table).

>>>
>>> The GUI DDU is invoked in silent mode on the live CD, to check for
>>> missing drivers and install them in the background. If additional
>>> drivers are still missing, the DDU pops up a window to tell the user,
>>> and to offer to display the GUI for manual action.
>>
>> This makes sense, thanks for the explanation. Why is there not an
>> equivalent check done in the background in the text-based installer?
> The lack of a non-windows environment to cleanly pop up a notification
> window is why the text-installer doesn't have a silent mode. Instead,
> the text-mode UI will be invoked by the start menu; similar to the start
> menu on S10, there is a menu item for running the installer, and there
> is another for running the text-mode DDU UI.

Okay.

Thanks,
-Seb

From Sebastien.Roy@Sun.COM Fri Mar  5 05:28:14 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o25DSDCV019700
	for <psarc-ext@sac.sfbay.sun.com>; Fri, 5 Mar 2010 05:28:13 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o25DSDSC008552
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Fri, 5 Mar 2010 06:28:13 -0700 (MST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYT00E0L9F1EI00@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Fri, 05 Mar 2010 05:28:13 -0800 (PST)
Received: from brmea-mail-1.sun.com ([192.18.98.31])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYT00CNI9F0K390@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Fri,
 05 Mar 2010 05:28:12 -0800 (PST)
Received: from fe-amer-10.sun.com ([192.18.109.80])
	by brmea-mail-1.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o25DSBqN011318	for
 <PSARC-ext@Sun.COM>; Fri, 05 Mar 2010 13:28:12 +0000 (GMT)
Received: from conversion-daemon.mail-amer.sun.com by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYT00M00952FS00@mail-amer.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Fri, 05 Mar 2010 06:28:11 -0700 (MST)
Received: from [192.168.1.3] ([unknown] [173.76.16.34])
 by mail-amer.sun.com (Sun Java(tm) System Messaging Server 7u2-7.04 64bit
 (built Jul  2 2009)) with ESMTPSA id <0KYT00HRC9EYTB20@mail-amer.sun.com> for
 PSARC-ext@Sun.COM (ORCPT PSARC-ext@Sun.COM); Fri,
 05 Mar 2010 06:28:11 -0700 (MST)
Date: Fri, 05 Mar 2010 08:28:10 -0500
From: Sebastien Roy <Sebastien.Roy@Sun.COM>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B90ABFD.60105@sun.com>
Sender: Sebastien.Roy@Sun.COM
To: Bill Yan <Xue-Yang.Yan@Sun.COM>
Cc: PSARC-ext@Sun.COM, Jack Schwartz <Jack.A.Schwartz@Sun.COM>,
        Frank Che <Frank.Che@Sun.COM>, "Tony.Hu@Sun.COM" <Tony.Hu@Sun.COM>,
        zhongyuan sun - Sun Microsystems - Beijing China <Zhongyuan.Sun@Sun.COM>
Message-id: <4B9106EA.3000506@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B90ABFD.60105@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100131
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 1881

Hi Bill,

On 03/ 5/10 02:00 AM, Bill Yan wrote:
> Interface | Stability | Description
> ------------------------+-------------------------+----------------------------------
>
> /usr/bin/ddu-text | Uncommitted | Command to invoke text mode UI
> ------------------------+-------------------------+----------------------------------
>
> /usr/bin/ddu --silent | Uncommitted | Command to invoke GUI ddu in
> silent mode
> ------------------------+-------------------------+-------------------------------
>
> ddu_devscan() | Consolidation private | DDU library API [2]
> ------------------------+-------------+----------------------------------
> ddu_package_lookup() | Consolidation private | DDU library API [2]
> ddu_build_repo_list() | |
> ddu_install_package() | |
> ------------------------+-------------------------+----------------------------------
>
> ddu_repo_object | Consolidation private | DDU library object [2]
> ddu_package_object | |
> ddu_dev_data | |
> ------------------------+-------------------------+----------------------------------

This looks good.  In addition, I'd like to see the python module name 
itself in the interface table, as this is the name programs will import 
(see separate disussion with Jack Schwartz).

> And we are going to add man pages for /usr/bin/ddu-text and /usr/bin/ddu
>

Excellent.

> Regarding the SVR4 and IPS package issue, Jack will help to address the
> issue and cite appropriate bugs to IPS team. Thanks a lot for Jack's help.

That is a step in the right direction, but I was hoping for something 
more specific before this case gets approved.  For example, a statement 
to the effect, "we've spoken with the IPS team, and they've committed to 
X to address the issue" or "we've spoken with the IPS team, and we will 
be modifying the architecture of this case by doing Y in order to play 
nice with IPS".

Thanks,
-Seb

From Xue-Yang.Yan@sun.com Sun Mar  7 22:07:20 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o2867KH4009060
	for <psarc-ext@sac.sfbay.sun.com>; Sun, 7 Mar 2010 22:07:20 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o2867JD5024361
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Sun, 7 Mar 2010 23:07:20 -0700 (MST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KYY00107907ZK00@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Sun, 07 Mar 2010 22:07:19 -0800 (PST)
Received: from sineb-mail-1.sun.com ([192.18.19.6])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KYY002B8906Z7B0@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Sun,
 07 Mar 2010 22:07:19 -0800 (PST)
Received: from fe-apac-06.sun.com
 (fe-apac-06.sun.com [192.18.19.177] (may be forged))
	by sineb-mail-1.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o2867IEp026823	for
 <PSARC-ext@Sun.COM>; Mon, 08 Mar 2010 06:07:18 +0000 (GMT)
Received: from conversion-daemon.mail-apac.sun.com by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KYY00K008PZ6300@mail-apac.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Mon, 08 Mar 2010 14:07:18 +0800 (SGT)
Received: from [129.158.218.53] ([unknown] [129.158.218.53])
 by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KYY00FW1904B3C0@mail-apac.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Mon, 08 Mar 2010 14:07:17 +0800 (SGT)
Date: Mon, 08 Mar 2010 14:04:22 +0800
From: Bill Yan <Xue-Yang.Yan@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B9106EA.3000506@sun.com>
Sender: Xue-Yang.Yan@sun.com
To: Sebastien Roy <Sebastien.Roy@sun.com>
Cc: PSARC-ext@sun.com, Jack Schwartz <Jack.A.Schwartz@sun.com>,
        Frank Che <Frank.Che@sun.com>, "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>,
        zhongyuan sun - Sun Microsystems - Beijing China <Zhongyuan.Sun@sun.com>
Message-id: <4B949366.4010400@sun.com>
MIME-version: 1.0
Content-type: text/plain; format=flowed; charset=UTF-8
Content-transfer-encoding: 8BIT
X-Accept-Language: zh-cn
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B90ABFD.60105@sun.com> <4B9106EA.3000506@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; zh-CN; rv:1.7) Gecko/20060627
Status: RO
Content-Length: 2082

Sebastien Roy 已写入:

> Hi Bill,
>
> On 03/ 5/10 02:00 AM, Bill Yan wrote:
>
>> Interface | Stability | Description
>> ------------------------+-------------------------+---------------------------------- 
>>
>>
>> /usr/bin/ddu-text | Uncommitted | Command to invoke text mode UI
>> ------------------------+-------------------------+---------------------------------- 
>>
>>
>> /usr/bin/ddu --silent | Uncommitted | Command to invoke GUI ddu in
>> silent mode
>> ------------------------+-------------------------+------------------------------- 
>>
>>
>> ddu_devscan() | Consolidation private | DDU library API [2]
>> ------------------------+-------------+---------------------------------- 
>>
>> ddu_package_lookup() | Consolidation private | DDU library API [2]
>> ddu_build_repo_list() | |
>> ddu_install_package() | |
>> ------------------------+-------------------------+---------------------------------- 
>>
>>
>> ddu_repo_object | Consolidation private | DDU library object [2]
>> ddu_package_object | |
>> ddu_dev_data | |
>> ------------------------+-------------------------+---------------------------------- 
>>
>
>
> This looks good.  In addition, I'd like to see the python module name 
> itself in the interface table, as this is the name programs will 
> import (see separate disussion with Jack Schwartz).

Ok, no problem.

>
>
>> And we are going to add man pages for /usr/bin/ddu-text and /usr/bin/ddu
>>
>
> Excellent.
>
>> Regarding the SVR4 and IPS package issue, Jack will help to address the
>> issue and cite appropriate bugs to IPS team. Thanks a lot for Jack's 
>> help.
>
>
> That is a step in the right direction, but I was hoping for something 
> more specific before this case gets approved.  For example, a 
> statement to the effect, "we've spoken with the IPS team, and they've 
> committed to X to address the issue" or "we've spoken with the IPS 
> team, and we will be modifying the architecture of this case by doing 
> Y in order to play nice with IPS".

I will let Jack to address this. Thanks for Jack's help.

>
> Thanks,
> -Seb



From Jack.A.Schwartz@sun.com Tue Mar  9 16:11:00 2010
Received: from sunmail2sca.sfbay.sun.com (sunmail2sca.SFBay.Sun.COM [129.145.155.234])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o2A0B0vp025662
	for <psarc-ext@sac.sfbay.sun.com>; Tue, 9 Mar 2010 16:11:00 -0800 (PST)
Received: from brm-avmta-1.central.sun.com (brm-avmta-1.Central.Sun.COM [129.147.4.11])
	by sunmail2sca.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o2A0Axse021949
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Tue, 9 Mar 2010 16:11:00 -0800 (PST)
Received: from pmxchannel-daemon.brm-avmta-1.central.sun.com by
 brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KZ100J07HUBT900@brm-avmta-1.central.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@Sun.COM); Tue, 09 Mar 2010 17:10:59 -0700 (MST)
Received: from sca-es-mail-2.sun.com ([192.18.43.133])
 by brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KZ100ILPHUATA00@brm-avmta-1.central.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@Sun.COM); Tue,
 09 Mar 2010 17:10:59 -0700 (MST)
Received: from fe-sfbay-09.sun.com ([192.18.43.129])
	by sca-es-mail-2.sun.com (8.13.7+Sun/8.12.9) with ESMTP id o2A0AwYH023471	for
 <PSARC-ext@Sun.COM>; Tue, 09 Mar 2010 16:10:58 -0800 (PST)
Received: from conversion-daemon.fe-sfbay-09.sun.com by fe-sfbay-09.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KZ100700HFOXW00@fe-sfbay-09.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Tue, 09 Mar 2010 16:10:58 -0800 (PST)
Received: from [129.146.226.171] ([unknown] [129.146.226.171])
 by fe-sfbay-09.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KZ100CZKHU9YB80@fe-sfbay-09.sun.com> for PSARC-ext@Sun.COM
 (ORCPT PSARC-ext@Sun.COM); Tue, 09 Mar 2010 16:10:58 -0800 (PST)
Date: Tue, 09 Mar 2010 16:09:54 -0800
From: Jack Schwartz <Jack.A.Schwartz@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B9106EA.3000506@sun.com>
Sender: Jack.A.Schwartz@sun.com
To: Sebastien Roy <sebastien.roy@sun.com>
Cc: Bill Yan <Xue-Yang.Yan@sun.com>, PSARC-ext@sun.com,
        Frank Che <Frank.Che@sun.com>, "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>,
        zhongyuan sun - Sun Microsystems - Beijing China <Zhongyuan.Sun@sun.com>
Message-id: <4B96E352.1020903@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B90ABFD.60105@sun.com> <4B9106EA.3000506@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100214
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 823

Hi Sebastien.

>> Regarding the SVR4 and IPS package issue, Jack will help to address the
>> issue and cite appropriate bugs to IPS team. Thanks a lot for Jack's 
>> help.
>
> That is a step in the right direction, but I was hoping for something 
> more specific before this case gets approved.  For example, a 
> statement to the effect, "we've spoken with the IPS team, and they've 
> committed to X to address the issue" or "we've spoken with the IPS 
> team, and we will be modifying the architecture of this case by doing 
> Y in order to play nice with IPS".
I have spoken with the IPS team about this issue, which is encapsulated 
as their bug 2434.  They have agreed to fix it.

I have also updated 2434 to make it clear that this PSARC case requires 
that it get fixed.

     Thanks,
     Jack
>
> Thanks,
> -Seb


From bart.smaalders@sun.com Tue Mar  9 16:23:09 2010
Received: from sunmail6brm.central.sun.com (sunmail6brm.Central.Sun.COM [129.147.4.169])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o2A0N914025977
	for <psarc-ext@sac.sfbay.sun.com>; Tue, 9 Mar 2010 16:23:09 -0800 (PST)
Received: from nwk-avmta-2.sfbay.sun.com (nwk-avmta-2.SFBay.Sun.COM [129.145.155.6])
	by sunmail6brm.central.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o2A0N5bH012206;
	Tue, 9 Mar 2010 18:23:06 -0600 (CST)
Received: from pmxchannel-daemon.nwk-avmta-2.sfbay.sun.com by
 nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KZ100K0PIEHZV00@nwk-avmta-2.sfbay.sun.com>; Tue,
 09 Mar 2010 16:23:05 -0800 (PST)
Received: from jurassic.Eng.Sun.COM ([129.146.17.59])
 by nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KZ100CRGIEHUD60@nwk-avmta-2.sfbay.sun.com>; Tue,
 09 Mar 2010 16:23:05 -0800 (PST)
Received: from [129.146.228.109] (cyber.SFBay.Sun.COM [129.146.228.109])
	by jurassic.Eng.Sun.COM (8.14.4+Sun/8.14.4) with ESMTP id o2A0N4Zb771849; Tue,
 09 Mar 2010 16:23:04 -0800 (PST)
Date: Tue, 09 Mar 2010 16:21:01 -0800
From: Bart Smaalders <bart.smaalders@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B96E352.1020903@sun.com>
To: Jack Schwartz <Jack.A.Schwartz@sun.com>
Cc: Sebastien Roy <sebastien.roy@sun.com>, Bill Yan <Xue-Yang.Yan@sun.com>,
        PSARC-ext@sun.com, Frank Che <Frank.Che@sun.com>,
        "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>,
        zhongyuan sun - Sun Microsystems - Beijing China <Zhongyuan.Sun@sun.com>
Message-id: <4B96E5ED.9010806@Sun.COM>
Organization: Sun Microsystems
MIME-version: 1.0
Content-type: text/plain; charset=ISO-8859-1; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B90ABFD.60105@sun.com> <4B9106EA.3000506@sun.com>
 <4B96E352.1020903@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100214
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 1083

On 03/09/10 16:09, Jack Schwartz wrote:
> Hi Sebastien.
>
>>> Regarding the SVR4 and IPS package issue, Jack will help to address the
>>> issue and cite appropriate bugs to IPS team. Thanks a lot for Jack's
>>> help.
>>
>> That is a step in the right direction, but I was hoping for something
>> more specific before this case gets approved. For example, a statement
>> to the effect, "we've spoken with the IPS team, and they've committed
>> to X to address the issue" or "we've spoken with the IPS team, and we
>> will be modifying the architecture of this case by doing Y in order to
>> play nice with IPS".
> I have spoken with the IPS team about this issue, which is encapsulated
> as their bug 2434. They have agreed to fix it.
>
> I have also updated 2434 to make it clear that this PSARC case requires
> that it get fixed.
>
> Thanks,
> Jack
>>
>> Thanks,
>> -Seb
>


With whom did you speak, Jack?

- Bart


-- 
Bart Smaalders			Solaris Kernel Performance
bart.smaalders@oracle.com	http://blogs.sun.com/barts
"You will contribute more with mercurial than with thunderbird."

From danek.duvall@sun.com Tue Mar  9 16:35:52 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o2A0ZqUL026033
	for <psarc-ext@sac.sfbay.sun.com>; Tue, 9 Mar 2010 16:35:52 -0800 (PST)
Received: from nwk-avmta-1.SFBay.Sun.COM (nwk-avmta-1.SFBay.Sun.COM [129.146.11.74])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o2A0Zp85005072
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Tue, 9 Mar 2010 17:35:52 -0700 (MST)
Received: from pmxchannel-daemon.nwk-avmta-1.sfbay.Sun.COM by
 nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KZ100L2RIZRW500@nwk-avmta-1.sfbay.Sun.COM> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Tue, 09 Mar 2010 16:35:51 -0800 (PST)
Received: from dm-sfbay-01.sfbay.sun.com ([129.145.155.118])
 by nwk-avmta-1.sfbay.Sun.COM
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KZ100FXEIZQZ010@nwk-avmta-1.sfbay.Sun.COM> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@sun.com); Tue,
 09 Mar 2010 16:35:50 -0800 (PST)
Received: from smelly.sfbay.sun.com (smelly.SFBay.Sun.COM [129.146.228.142])
	by dm-sfbay-01.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4)
 with ESMTP id o2A0ZlJ5029886; Tue, 09 Mar 2010 16:35:47 -0800 (PST)
Received: from smelly.sfbay.sun.com (smelly [127.0.0.1])
	by smelly.sfbay.sun.com (8.14.4+Sun/8.14.4) with ESMTP id o2A0Zlgt005274; Tue,
 09 Mar 2010 16:35:47 -0800 (PST)
Received: (from dduvall@localhost)
	by smelly.sfbay.sun.com (8.14.4+Sun/8.14.4/Submit) id o2A0ZkVF005273; Tue,
 09 Mar 2010 16:35:46 -0800 (PST)
Date: Tue, 09 Mar 2010 16:35:46 -0800
From: Danek Duvall <danek.duvall@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B96E352.1020903@sun.com>
To: Jack Schwartz <Jack.A.Schwartz@sun.com>
Cc: Sebastien Roy <sebastien.roy@sun.com>, Bill Yan <Xue-Yang.Yan@sun.com>,
        PSARC-ext@sun.com, Frank Che <Frank.Che@sun.com>,
        "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>,
        zhongyuan sun - Sun Microsystems - Beijing China <Zhongyuan.Sun@sun.com>
Message-id: <20100310003546.GQ1033@smelly.SFBay.Sun.COM>
MIME-version: 1.0
Content-type: text/plain; charset=us-ascii
Content-transfer-encoding: 7BIT
Content-disposition: inline
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B90ABFD.60105@sun.com> <4B9106EA.3000506@sun.com>
 <4B96E352.1020903@sun.com>
User-Agent: Mutt/1.5.20 (2009-06-23)
Status: RO
Content-Length: 862

Jack Schwartz wrote:

> Hi Sebastien.
> 
> >>Regarding the SVR4 and IPS package issue, Jack will help to address the
> >>issue and cite appropriate bugs to IPS team. Thanks a lot for
> >>Jack's help.
> >
> >That is a step in the right direction, but I was hoping for
> >something more specific before this case gets approved.  For
> >example, a statement to the effect, "we've spoken with the IPS
> >team, and they've committed to X to address the issue" or "we've
> >spoken with the IPS team, and we will be modifying the architecture
> >of this case by doing Y in order to play nice with IPS".
> I have spoken with the IPS team about this issue, which is
> encapsulated as their bug 2434.  They have agreed to fix it.

FWIW, we (I) did not agree to fix bug 2434; we simply agreed to work with
the DDU team to make sure that their requirements were met.

Danek

From Sebastien.Roy@Sun.COM Tue Mar  9 16:55:05 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o2A0t4gF026724
	for <psarc-ext@sac.sfbay.sun.com>; Tue, 9 Mar 2010 16:55:04 -0800 (PST)
Received: from nwk-avmta-2.sfbay.sun.com (nwk-avmta-2.SFBay.Sun.COM [129.145.155.6])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o2A0t454016501
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Tue, 9 Mar 2010 17:55:04 -0700 (MST)
Received: from pmxchannel-daemon.nwk-avmta-2.sfbay.sun.com by
 nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KZ100001JVS1000@nwk-avmta-2.sfbay.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Tue, 09 Mar 2010 16:55:04 -0800 (PST)
Received: from brmea-mail-4.sun.com ([192.18.98.36])
 by nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KZ100CJ7JVRUE80@nwk-avmta-2.sfbay.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@sun.com); Tue,
 09 Mar 2010 16:55:03 -0800 (PST)
Received: from fe-amer-09.sun.com ([192.18.109.79])
	by brmea-mail-4.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o2A0t32Q025869	for
 <PSARC-ext@sun.com>; Wed, 10 Mar 2010 00:55:03 +0000 (GMT)
Received: from conversion-daemon.mail-amer.sun.com by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KZ100B00JH1YN00@mail-amer.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Tue, 09 Mar 2010 17:55:03 -0700 (MST)
Received: from [192.168.1.3] ([unknown] [173.76.16.34])
 by mail-amer.sun.com (Sun Java(tm) System Messaging Server 7u2-7.04 64bit
 (built Jul  2 2009)) with ESMTPSA id <0KZ100FU3JVQSG00@mail-amer.sun.com>; Tue,
 09 Mar 2010 17:55:03 -0700 (MST)
Date: Tue, 09 Mar 2010 19:55:02 -0500
From: Sebastien Roy <Sebastien.Roy@Sun.COM>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <20100310003546.GQ1033@smelly.SFBay.Sun.COM>
Sender: Sebastien.Roy@Sun.COM
To: Danek Duvall <Danek.Duvall@Sun.COM>
Cc: Jack Schwartz <Jack.A.Schwartz@Sun.COM>, Bill Yan <Xue-Yang.Yan@Sun.COM>,
        PSARC-ext@Sun.COM, Frank Che <Frank.Che@Sun.COM>,
        "Tony.Hu@Sun.COM" <Tony.Hu@Sun.COM>,
        zhongyuan sun - Sun Microsystems - Beijing China <Zhongyuan.Sun@Sun.COM>
Message-id: <4B96EDE6.40307@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B90ABFD.60105@sun.com> <4B9106EA.3000506@sun.com>
 <4B96E352.1020903@sun.com> <20100310003546.GQ1033@smelly.SFBay.Sun.COM>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100214
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 617

On 03/ 9/10 07:35 PM, Danek Duvall wrote:
> Jack Schwartz wrote:
>> I have spoken with the IPS team about this issue, which is
>> encapsulated as their bug 2434.  They have agreed to fix it.
>
> FWIW, we (I) did not agree to fix bug 2434; we simply agreed to work with
> the DDU team to make sure that their requirements were met.

Okay, I'm happy with this outcome.  The important thing is that the 
right people are talking and that this case can be integrated as 
specified.  I have no further issues with this case (modulo the pending 
changes to the interface table based on my previous comments).

Thanks,
-Seb

From Frank.Che@sun.com Tue Mar  9 23:57:05 2010
Received: from sunmail3mpk.sfbay.sun.com (sunmail3mpk.SFBay.Sun.COM [129.146.11.52])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o2A7v5S3002474
	for <psarc-ext@sac.sfbay.sun.com>; Tue, 9 Mar 2010 23:57:05 -0800 (PST)
Received: from brm-avmta-1.central.sun.com (brm-avmta-1.Central.Sun.COM [129.147.4.11])
	by sunmail3mpk.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o2A7v4xg012888
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Tue, 9 Mar 2010 23:57:05 -0800 (PST)
Received: from pmxchannel-daemon.brm-avmta-1.central.sun.com by
 brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KZ20020H3F4XB00@brm-avmta-1.central.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Wed, 10 Mar 2010 00:57:04 -0700 (MST)
Received: from sineb-mail-2.sun.com ([192.18.19.7])
 by brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KZ2009IF3F2DBE0@brm-avmta-1.central.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@sun.com); Wed,
 10 Mar 2010 00:57:03 -0700 (MST)
Received: from fe-apac-06.sun.com
 (fe-apac-06.sun.com [192.18.19.177] (may be forged))
	by sineb-mail-2.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o2A7v13W026550	for
 <PSARC-ext@sun.com>; Wed, 10 Mar 2010 07:57:01 +0000 (GMT)
Received: from conversion-daemon.mail-apac.sun.com by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KZ200G00357XQ00@mail-apac.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Wed, 10 Mar 2010 15:57:01 +0800 (SGT)
Received: from [129.158.218.60] ([unknown] [129.158.218.60])
 by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KZ2009P23F0CC40@mail-apac.sun.com>; Wed,
 10 Mar 2010 15:57:01 +0800 (SGT)
Date: Wed, 10 Mar 2010 15:56:50 +0800
From: Frank Che <Frank.Che@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B96EDE6.40307@sun.com>
Sender: Frank.Che@sun.com
To: Sebastien Roy <Sebastien.Roy@sun.com>
Cc: Danek Duvall <Danek.Duvall@sun.com>,
        Jack Schwartz <Jack.A.Schwartz@sun.com>,
        Bill Yan <Xue-Yang.Yan@sun.com>, PSARC-ext@sun.com,
        "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>,
        zhongyuan sun - Sun Microsystems - Beijing China <Zhongyuan.Sun@sun.com>
Message-id: <4B9750C2.9070108@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B90ABFD.60105@sun.com> <4B9106EA.3000506@sun.com>
 <4B96E352.1020903@sun.com> <20100310003546.GQ1033@smelly.SFBay.Sun.COM>
 <4B96EDE6.40307@sun.com>
User-Agent: Thunderbird 2.0.0.19 (X11/20090218)
Status: RO
Content-Length: 850

The project team has updated the exported interface table according to 
the review feedbacks. I have just updated file 'spec.txt' in the case 
directory to reflect this change.

-Frank

Sebastien Roy wrote:
> On 03/ 9/10 07:35 PM, Danek Duvall wrote:
>> Jack Schwartz wrote:
>>> I have spoken with the IPS team about this issue, which is
>>> encapsulated as their bug 2434.  They have agreed to fix it.
>>
>> FWIW, we (I) did not agree to fix bug 2434; we simply agreed to work 
>> with
>> the DDU team to make sure that their requirements were met.
>
> Okay, I'm happy with this outcome.  The important thing is that the 
> right people are talking and that this case can be integrated as 
> specified.  I have no further issues with this case (modulo the 
> pending changes to the interface table based on my previous comments).
>
> Thanks,
> -Seb

From sebastien.roy@oracle.com Wed Mar 10 07:53:48 2010
Received: from sunmail2sca.sfbay.sun.com (sunmail2sca.SFBay.Sun.COM [129.145.155.234])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o2AFrmpv027374
	for <psarc-ext@sac.sfbay.sun.com>; Wed, 10 Mar 2010 07:53:48 -0800 (PST)
Received: from nwk-avmta-2.sfbay.sun.com (nwk-avmta-2.SFBay.Sun.COM [129.145.155.6])
	by sunmail2sca.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o2AFrjE3001782;
	Wed, 10 Mar 2010 07:53:46 -0800 (PST)
Received: from pmxchannel-daemon.nwk-avmta-2.sfbay.sun.com by
 nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KZ200D0LPHLDK00@nwk-avmta-2.sfbay.sun.com>; Wed,
 10 Mar 2010 07:53:45 -0800 (PST)
Received: from sca-ea-mail-3.sun.com ([192.18.43.21])
 by nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KZ20050GPHHHWA0@nwk-avmta-2.sfbay.sun.com>; Wed,
 10 Mar 2010 07:53:41 -0800 (PST)
Received: from acsinet15.oracle.com (acsinet15.oracle.com [141.146.126.227])
	by sca-ea-mail-3.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o2AFreoJ024705;
 Wed, 10 Mar 2010 15:53:41 +0000 (GMT)
Received: from acsmt354.oracle.com (acsmt354.oracle.com [141.146.40.154])
	by acsinet15.oracle.com (Switch-3.4.2/Switch-3.4.1)
 with ESMTP id o2AFrXbV028568; Wed, 10 Mar 2010 15:53:33 +0000 (GMT)
Received: from abhmt004.oracle.com by acsmt355.oracle.com	with ESMTP id
 75502781268236408; Wed, 10 Mar 2010 07:53:28 -0800
Received: from [129.148.174.103] (/129.148.174.103)
	by default (Oracle Beehive Gateway v4.0)	with ESMTP ; Wed,
 10 Mar 2010 07:53:28 -0800
Date: Wed, 10 Mar 2010 10:50:41 -0500
From: Sebastien Roy <sebastien.roy@oracle.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B9750C2.9070108@sun.com>
To: Frank Che <Frank.Che@sun.com>
Cc: Sebastien Roy <Sebastien.Roy@sun.com>, "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>,
        PSARC-ext@sun.com,
        zhongyuan sun - Sun Microsystems - Beijing China
 <Zhongyuan.Sun@sun.com>,
        Jack Schwartz <Jack.A.Schwartz@sun.com>,
        Bill Yan <Xue-Yang.Yan@sun.com>
Message-id: <4B97BFD1.8070704@oracle.com>
MIME-version: 1.0
Content-type: text/plain; charset=ISO-8859-1; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
X-Source-IP: acsmt354.oracle.com [141.146.40.154]
X-Auth-Type: Internal IP
X-CT-RefId: str=0001.0A090207.4B97C084.009B:SCFMA4539814,ss=1,fgs=0
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B90ABFD.60105@sun.com> <4B9106EA.3000506@sun.com>
 <4B96E352.1020903@sun.com> <20100310003546.GQ1033@smelly.SFBay.Sun.COM>
 <4B96EDE6.40307@sun.com> <4B9750C2.9070108@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100214
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 2727

On 03/10/10 02:56 AM, Frank Che wrote:
> The project team has updated the exported interface table according to
> the review feedbacks. I have just updated file 'spec.txt' in the case
> directory to reflect this change.

I'm sorry to keep harping on this point, but the python module and/or 
package names are still not included as an exported interface, and that 
seems like an important detail.  Programs need to "import <module>" to 
get at this stuff, right?  What is that?  Assuming that we resolve this 
issue, I give the case a +1.

That said, looking more closely at the API, I have some additional 
feedback.  It's a Consolidation Private API and these are all 
superficial issues, so I don't feel strongly about the below issues.

* There's no need to prefix all of your symbols with "ddu_".  Don't you 
need to access your base methods using <module>.<function>() anyway? 
E.g., ddu.devscan() seems cleaner than ddu.ddu_devscan().

* The class names all have an "_object" suffix, which is strange. 
Here's are suggested changes:
   - ddu_dev_data_object  ->  device
   - ddu_repo_object      ->  repository
   - ddu_package_object   ->  package

* Your accessor functions (get_location(), get_name(), and get_type()) 
are unnecessary, and should be attribute references (e.g., 
package.location, package.name, and package.type).

* The ddu_install_package() function could easily be a method of the 
package class.  Instead of passing a package object to the 
ddu_install_package() function, it would be more logical to simply call 
an install method of an instantiated package object.

* I have a similar comment regarding the ddu_package_lookup() function 
taking a device object as an argument.  You're looking up the package 
object that is associated with a device, so the package lookup function 
should really simply be a method of the package object.

To put this together, you currently essentially have (distilled from you 
code snipet example):

         device_list[] = ddu_devscan(return_missing_only, "all")
         repos[] = ddu_build_repo_list(list of (name, URL) tuples)
         for device in device_list:
                 # Look up the package for the given device.
                 ddu_package_obj =
                     ddu_package_lookup(device, repos[])

                 ddu_install_package(ddu_package_obj, "/")
                 ...

This would become something like:

         devices[] = ddu.device_scan(return_missing_only, "all")
         repos[] = ddu.build_repo_list(list of (name, URL) tuples)
         for device in devices:
                 # Look up the package for the given device.
                 ddupkg = device.getpackage(repos[])
                 ddupkg.install("/")

-Seb

From Sebastien.Roy@sun.com Wed Mar 10 08:40:06 2010
Received: from newsunmail1brm.central.sun.com (newsunmail1brm.Central.Sun.COM [129.147.62.245])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o2AGe5YZ029268
	for <psarc-ext@sac.sfbay.sun.com>; Wed, 10 Mar 2010 08:40:05 -0800 (PST)
Received: from nwk-avmta-2.sfbay.sun.com (nwk-avmta-2.SFBay.Sun.COM [129.145.155.6])
	by newsunmail1brm.central.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.4) with ESMTP id o2AGe2fO023877
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Wed, 10 Mar 2010 09:40:05 -0700 (MST)
Received: from pmxchannel-daemon.nwk-avmta-2.sfbay.sun.com by
 nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KZ200F1VRMRUK00@nwk-avmta-2.sfbay.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Wed, 10 Mar 2010 08:40:03 -0800 (PST)
Received: from brmea-mail-4.sun.com ([192.18.98.36])
 by nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KZ200FI5RMRIY10@nwk-avmta-2.sfbay.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@sun.com); Wed,
 10 Mar 2010 08:40:03 -0800 (PST)
Received: from fe-amer-10.sun.com ([192.18.109.80])
	by brmea-mail-4.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o2AGe2Ax012567	for
 <PSARC-ext@sun.com>; Wed, 10 Mar 2010 16:40:02 +0000 (GMT)
Received: from conversion-daemon.mail-amer.sun.com by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KZ200F00RDWRF00@mail-amer.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Wed, 10 Mar 2010 09:40:02 -0700 (MST)
Received: from [129.148.174.103] ([unknown] [129.148.174.103])
 by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KZ200ETNRMFYK70@mail-amer.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Wed, 10 Mar 2010 09:39:52 -0700 (MST)
Date: Wed, 10 Mar 2010 11:37:02 -0500
From: Sebastien Roy <Sebastien.Roy@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B97BFD1.8070704@oracle.com>
Sender: Sebastien.Roy@sun.com
To: Sebastien Roy <sebastien.roy@oracle.com>
Cc: Frank Che <Frank.Che@sun.com>, "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>,
        PSARC-ext@sun.com,
        zhongyuan sun - Sun Microsystems - Beijing China
 <Zhongyuan.Sun@sun.com>,
        Jack Schwartz <Jack.A.Schwartz@sun.com>,
        Bill Yan <Xue-Yang.Yan@sun.com>
Message-id: <4B97CAAE.4070805@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B90ABFD.60105@sun.com> <4B9106EA.3000506@sun.com>
 <4B96E352.1020903@sun.com> <20100310003546.GQ1033@smelly.SFBay.Sun.COM>
 <4B96EDE6.40307@sun.com> <4B9750C2.9070108@sun.com>
 <4B97BFD1.8070704@oracle.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100214
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 369

On 03/10/10 10:50 AM, Sebastien Roy wrote:
> * I have a similar comment regarding the ddu_package_lookup() function
> taking a device object as an argument. You're looking up the package
> object that is associated with a device, so the package lookup function
> should really simply be a method of the package object.

That's a typo, I meant device object above.
-Seb

From Jack.A.Schwartz@sun.com Wed Mar 10 12:53:23 2010
Received: from sunmail2sca.sfbay.sun.com (sunmail2sca.SFBay.Sun.COM [129.145.155.234])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o2AKrNxl008705
	for <psarc-ext@sac.sfbay.sun.com>; Wed, 10 Mar 2010 12:53:23 -0800 (PST)
Received: from nwk-avmta-2.sfbay.sun.com (nwk-avmta-2.SFBay.Sun.COM [129.145.155.6])
	by sunmail2sca.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o2AKrMFJ009816
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Wed, 10 Mar 2010 12:53:23 -0800 (PST)
Received: from pmxchannel-daemon.nwk-avmta-2.sfbay.sun.com by
 nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KZ3007253CYXS00@nwk-avmta-2.sfbay.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Wed, 10 Mar 2010 12:53:22 -0800 (PST)
Received: from sca-es-mail-2.sun.com ([192.18.43.133])
 by nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KZ3001RU3CYOP50@nwk-avmta-2.sfbay.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@sun.com); Wed,
 10 Mar 2010 12:53:22 -0800 (PST)
Received: from fe-sfbay-09.sun.com ([192.18.43.129])
	by sca-es-mail-2.sun.com (8.13.7+Sun/8.12.9) with ESMTP id o2AKrMS3021826	for
 <PSARC-ext@sun.com>; Wed, 10 Mar 2010 12:53:22 -0800 (PST)
Received: from conversion-daemon.fe-sfbay-09.sun.com by fe-sfbay-09.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KZ300G0036OKI00@fe-sfbay-09.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Wed, 10 Mar 2010 12:53:19 -0800 (PST)
Received: from [129.146.226.171] ([unknown] [129.146.226.171])
 by fe-sfbay-09.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KZ300E553CU1Q50@fe-sfbay-09.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Wed, 10 Mar 2010 12:53:18 -0800 (PST)
Date: Wed, 10 Mar 2010 12:52:14 -0800
From: Jack Schwartz <Jack.A.Schwartz@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B97BFD1.8070704@oracle.com>
Sender: Jack.A.Schwartz@sun.com
To: Sebastien Roy <sebastien.roy@oracle.com>
Cc: Frank Che <Frank.Che@sun.com>, Sebastien Roy <Sebastien.Roy@sun.com>,
        "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>, PSARC-ext@sun.com,
        zhongyuan sun - Sun Microsystems - Beijing China <Zhongyuan.Sun@sun.com>,
        Bill Yan <Xue-Yang.Yan@sun.com>
Message-id: <4B98067E.7030700@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B90ABFD.60105@sun.com> <4B9106EA.3000506@sun.com>
 <4B96E352.1020903@sun.com> <20100310003546.GQ1033@smelly.SFBay.Sun.COM>
 <4B96EDE6.40307@sun.com> <4B9750C2.9070108@sun.com>
 <4B97BFD1.8070704@oracle.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100214
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 4308

Hi Sebastien.

On 03/10/10 07:50 AM, Sebastien Roy wrote:
> On 03/10/10 02:56 AM, Frank Che wrote:
>> The project team has updated the exported interface table according to
>> the review feedbacks. I have just updated file 'spec.txt' in the case
>> directory to reflect this change.
>
> I'm sorry to keep harping on this point, but the python module and/or 
> package names are still not included as an exported interface, and 
> that seems like an important detail.  Programs need to "import 
> <module>" to get at this stuff, right?  What is that?  Assuming that 
> we resolve this issue, I give the case a +1.
Since I wrote the specs that the below issues deal with (with the help 
of the DDU team, of course), I feel it is appropriate for me to respond...

For convenience regarding modifying the case documents, current 
filenames and the classes/functions they contain are:

ddu_devdata.py :
     ddu_dev_data class
ddu_errors.py:
     contains all DDU-specific exception classes.
ddu_function.py:
     ddu_build_repo_list()
     ddu_devscan()
     ddu_package_lookup()
     ddu_install_package()
ddu_package.py:
     ddu_package_object class
ddu_repo.py:
     ddu_repo_object class
> That said, looking more closely at the API, I have some additional 
> feedback.  It's a Consolidation Private API and these are all 
> superficial issues, so I don't feel strongly about the below issues.
>
> * There's no need to prefix all of your symbols with "ddu_".  Don't 
> you need to access your base methods using <module>.<function>() 
> anyway? E.g., ddu.devscan() seems cleaner than ddu.ddu_devscan().
Yes and no.  One can access methods as you say, but more commonly, one 
says at the top of a python module:

from DDU.ddu_function.py import ddu_package_lookup

and then uses ddu_package_lookup henceforth in that module without a 
prefix.  No need to muddy up the code unnecessarily  with <module> every 
time you call a function.

This is standard practice and what the DDU code does.

I think the function names can stay as they are.
>
> * The class names all have an "_object" suffix, which is strange. 
> Here's are suggested changes:
>   - ddu_dev_data_object  ->  device
>   - ddu_repo_object      ->  repository
>   - ddu_package_object   ->  package
I'd like to leave this for now.
>
> * Your accessor functions (get_location(), get_name(), and get_type()) 
> are unnecessary, and should be attribute references (e.g., 
> package.location, package.name, and package.type).
OK.  This needs to be changed in the spec, but the code is already like 
this.
>
> * The ddu_install_package() function could easily be a method of the 
> package class.  Instead of passing a package object to the 
> ddu_install_package() function, it would be more logical to simply 
> call an install method of an instantiated package object.
This is a good suggestion and I'd like to follow up on it at a later date.
>
> * I have a similar comment regarding the ddu_package_lookup() function 
> taking a device object as an argument.  You're looking up the package 
> object that is associated with a device, so the package lookup 
> function should really simply be a method of the package object.
>
> To put this together, you currently essentially have (distilled from 
> you code snipet example):
>
>         device_list[] = ddu_devscan(return_missing_only, "all")
>         repos[] = ddu_build_repo_list(list of (name, URL) tuples)
>         for device in device_list:
>                 # Look up the package for the given device.
>                 ddu_package_obj =
>                     ddu_package_lookup(device, repos[])
>
>                 ddu_install_package(ddu_package_obj, "/")
>                 ...
>
> This would become something like:
>
>         devices[] = ddu.device_scan(return_missing_only, "all")
>         repos[] = ddu.build_repo_list(list of (name, URL) tuples)
>         for device in devices:
>                 # Look up the package for the given device.
>                 ddupkg = device.getpackage(repos[])
>                 ddupkg.install("/")
This is another good suggestion that I'd like to follow up on it at a 
later date.

I plan on filing a bug for these changes so they don't get dropped on 
the floor, after I take the DDU team's input (tonight).

     thanks,
     Jack
>
> -Seb


From Sebastien.Roy@Sun.COM Wed Mar 10 13:37:04 2010
Received: from sunmail3mpk.sfbay.sun.com (sunmail3mpk.SFBay.Sun.COM [129.146.11.52])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o2ALb4Qq009829
	for <psarc-ext@sac.sfbay.sun.com>; Wed, 10 Mar 2010 13:37:04 -0800 (PST)
Received: from brm-avmta-1.central.sun.com (brm-avmta-1.Central.Sun.COM [129.147.4.11])
	by sunmail3mpk.sfbay.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o2ALb1hE000752
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Wed, 10 Mar 2010 13:37:04 -0800 (PST)
Received: from pmxchannel-daemon.brm-avmta-1.central.sun.com by
 brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KZ300I0X5DR4M00@brm-avmta-1.central.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Wed, 10 Mar 2010 14:37:03 -0700 (MST)
Received: from brmea-mail-1.sun.com ([192.18.98.31])
 by brm-avmta-1.central.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KZ3008LO5DR8W50@brm-avmta-1.central.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@sun.com); Wed,
 10 Mar 2010 14:37:03 -0700 (MST)
Received: from fe-amer-09.sun.com ([192.18.109.79])
	by brmea-mail-1.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o2ALb34C007660	for
 <PSARC-ext@sun.com>; Wed, 10 Mar 2010 21:37:03 +0000 (GMT)
Received: from conversion-daemon.mail-amer.sun.com by mail-amer.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KZ30030052OG400@mail-amer.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Wed, 10 Mar 2010 14:37:03 -0700 (MST)
Received: from [192.168.1.3] ([unknown] [173.76.16.34])
 by mail-amer.sun.com (Sun Java(tm) System Messaging Server 7u2-7.04 64bit
 (built Jul  2 2009)) with ESMTPSA id <0KZ3006Q55DOTD60@mail-amer.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@sun.com); Wed,
 10 Mar 2010 14:37:01 -0700 (MST)
Date: Wed, 10 Mar 2010 16:36:59 -0500
From: Sebastien Roy <Sebastien.Roy@Sun.COM>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B98067E.7030700@sun.com>
Sender: Sebastien.Roy@Sun.COM
To: Jack Schwartz <Jack.A.Schwartz@Sun.COM>
Cc: Sebastien Roy <sebastien.roy@oracle.com>, Frank Che <Frank.Che@Sun.COM>,
        "Tony.Hu@Sun.COM" <Tony.Hu@Sun.COM>, PSARC-ext@Sun.COM,
        zhongyuan sun - Sun Microsystems - Beijing China <Zhongyuan.Sun@Sun.COM>,
        Bill Yan <Xue-Yang.Yan@Sun.COM>
Message-id: <4B9810FB.7090207@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B90ABFD.60105@sun.com> <4B9106EA.3000506@sun.com>
 <4B96E352.1020903@sun.com> <20100310003546.GQ1033@smelly.SFBay.Sun.COM>
 <4B96EDE6.40307@sun.com> <4B9750C2.9070108@sun.com>
 <4B97BFD1.8070704@oracle.com> <4B98067E.7030700@sun.com>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.1.7) Gecko/20100214
 Lightning/1.0b1 Thunderbird/3.0.1
Status: RO
Content-Length: 4656

Hi Jack,

On 03/10/10 03:52 PM, Jack Schwartz wrote:
> Hi Sebastien.
>
> On 03/10/10 07:50 AM, Sebastien Roy wrote:
>> On 03/10/10 02:56 AM, Frank Che wrote:
>>> The project team has updated the exported interface table according to
>>> the review feedbacks. I have just updated file 'spec.txt' in the case
>>> directory to reflect this change.
>>
>> I'm sorry to keep harping on this point, but the python module and/or
>> package names are still not included as an exported interface, and
>> that seems like an important detail. Programs need to "import
>> <module>" to get at this stuff, right? What is that? Assuming that we
>> resolve this issue, I give the case a +1.
> Since I wrote the specs that the below issues deal with (with the help
> of the DDU team, of course), I feel it is appropriate for me to respond...
>
> For convenience regarding modifying the case documents, current
> filenames and the classes/functions they contain are:
>
> ddu_devdata.py :
> ddu_dev_data class
> ddu_errors.py:
> contains all DDU-specific exception classes.
> ddu_function.py:
> ddu_build_repo_list()
> ddu_devscan()
> ddu_package_lookup()
> ddu_install_package()
> ddu_package.py:
> ddu_package_object class
> ddu_repo.py:
> ddu_repo_object class
>> That said, looking more closely at the API, I have some additional
>> feedback. It's a Consolidation Private API and these are all
>> superficial issues, so I don't feel strongly about the below issues.
>>
>> * There's no need to prefix all of your symbols with "ddu_". Don't you
>> need to access your base methods using <module>.<function>() anyway?
>> E.g., ddu.devscan() seems cleaner than ddu.ddu_devscan().
> Yes and no. One can access methods as you say, but more commonly, one
> says at the top of a python module:
>
> from DDU.ddu_function.py import ddu_package_lookup
>
> and then uses ddu_package_lookup henceforth in that module without a
> prefix. No need to muddy up the code unnecessarily with <module> every
> time you call a function.

It's your call, but honestly, your argument doesn't seem logical 
(regarding muddying up the code) given the difference is between 
ddu_package_lookup() and ddu.package_lookup()...  The difference is a 
single character.  It seems like you're reinventing a namespace 
convention that's already provided for you by the python module 
framework.  Anyway, it's up to you and your code-reviewers, I don't feel 
strongly about it.

>
> This is standard practice and what the DDU code does.
>
> I think the function names can stay as they are.
>>
>> * The class names all have an "_object" suffix, which is strange.
>> Here's are suggested changes:
>> - ddu_dev_data_object -> device
>> - ddu_repo_object -> repository
>> - ddu_package_object -> package
> I'd like to leave this for now.
>>
>> * Your accessor functions (get_location(), get_name(), and get_type())
>> are unnecessary, and should be attribute references (e.g.,
>> package.location, package.name, and package.type).
> OK. This needs to be changed in the spec, but the code is already like
> this.
>>
>> * The ddu_install_package() function could easily be a method of the
>> package class. Instead of passing a package object to the
>> ddu_install_package() function, it would be more logical to simply
>> call an install method of an instantiated package object.
> This is a good suggestion and I'd like to follow up on it at a later date.
>>
>> * I have a similar comment regarding the ddu_package_lookup() function
>> taking a device object as an argument. You're looking up the package
>> object that is associated with a device, so the package lookup
>> function should really simply be a method of the package object.
>>
>> To put this together, you currently essentially have (distilled from
>> you code snipet example):
>>
>> device_list[] = ddu_devscan(return_missing_only, "all")
>> repos[] = ddu_build_repo_list(list of (name, URL) tuples)
>> for device in device_list:
>> # Look up the package for the given device.
>> ddu_package_obj =
>> ddu_package_lookup(device, repos[])
>>
>> ddu_install_package(ddu_package_obj, "/")
>> ...
>>
>> This would become something like:
>>
>> devices[] = ddu.device_scan(return_missing_only, "all")
>> repos[] = ddu.build_repo_list(list of (name, URL) tuples)
>> for device in devices:
>> # Look up the package for the given device.
>> ddupkg = device.getpackage(repos[])
>> ddupkg.install("/")
> This is another good suggestion that I'd like to follow up on it at a
> later date.
>
> I plan on filing a bug for these changes so they don't get dropped on
> the floor, after I take the DDU team's input (tonight).

That's all fine by me.

Thanks,
-Seb

From Frank.Che@sun.com Wed Mar 10 17:49:52 2010
Received: from sunmail6brm.central.sun.com (sunmail6brm.Central.Sun.COM [129.147.4.169])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id o2B1nqtp014884
	for <psarc-ext@sac.sfbay.sun.com>; Wed, 10 Mar 2010 17:49:52 -0800 (PST)
Received: from nwk-avmta-2.sfbay.sun.com (nwk-avmta-2.SFBay.Sun.COM [129.145.155.6])
	by sunmail6brm.central.sun.com (8.13.8+Sun/8.13.8/ENSMAIL,v2.4) with ESMTP id o2B1npXh007337
	for <@sunmail2sca.sfbay.sun.com:PSARC-ext@sun.com>; Wed, 10 Mar 2010 19:49:52 -0600 (CST)
Received: from pmxchannel-daemon.nwk-avmta-2.sfbay.sun.com by
 nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0KZ300407H33ER00@nwk-avmta-2.sfbay.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Wed, 10 Mar 2010 17:49:51 -0800 (PST)
Received: from sineb-mail-1.sun.com ([192.18.19.6])
 by nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0KZ300N9GH312950@nwk-avmta-2.sfbay.sun.com> for
 PSARC-ext@sun.com (ORCPT PSARC-ext@sun.com); Wed,
 10 Mar 2010 17:49:50 -0800 (PST)
Received: from fe-apac-06.sun.com
 (fe-apac-06.sun.com [192.18.19.177] (may be forged))
	by sineb-mail-1.sun.com (8.13.6+Sun/8.12.9) with ESMTP id o2B1nnIn006626	for
 <PSARC-ext@sun.com>; Thu, 11 Mar 2010 01:49:49 +0000 (GMT)
Received: from conversion-daemon.mail-apac.sun.com by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 id <0KZ300700GWH3200@mail-apac.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Thu, 11 Mar 2010 09:49:49 +0800 (SGT)
Received: from [129.158.218.60] ([unknown] [129.158.218.60])
 by mail-apac.sun.com
 (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul  2 2009))
 with ESMTPSA id <0KZ3009AZH2HCCJ0@mail-apac.sun.com> for PSARC-ext@sun.com
 (ORCPT PSARC-ext@sun.com); Thu, 11 Mar 2010 09:49:49 +0800 (SGT)
Date: Thu, 11 Mar 2010 09:49:20 +0800
From: Frank Che <Frank.Che@sun.com>
Subject: Re: Add Text Mode UI and Common Library to Device Driver Utility
 [PSARC/2009/602 FastTrack timeout 03/05/2010]
In-reply-to: <4B9810FB.7090207@sun.com>
Sender: Frank.Che@sun.com
To: PSARC-ext@sun.com
Cc: Jack Schwartz <Jack.A.Schwartz@sun.com>,
        "Tony.Hu@Sun.COM" <Tony.Hu@sun.com>,
        zhongyuan sun - Sun Microsystems - Beijing China <Zhongyuan.Sun@sun.com>,
        Bill Yan <Xue-Yang.Yan@sun.com>
Message-id: <4B984C20.70802@sun.com>
MIME-version: 1.0
Content-type: text/plain; CHARSET=US-ASCII; format=flowed
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.4.1.325704
References: <4AF3AF60.7050801@sun.com> <4B876781.4090007@sun.com>
 <4B8E7F92.6050605@sun.com> <4B8ED3C7.5010601@sun.com>
 <4B8FDD0F.6040603@sun.com> <4B90ABFD.60105@sun.com> <4B9106EA.3000506@sun.com>
 <4B96E352.1020903@sun.com> <20100310003546.GQ1033@smelly.SFBay.Sun.COM>
 <4B96EDE6.40307@sun.com> <4B9750C2.9070108@sun.com>
 <4B97BFD1.8070704@oracle.com> <4B98067E.7030700@sun.com>
 <4B9810FB.7090207@sun.com>
User-Agent: Thunderbird 2.0.0.19 (X11/20090218)
Status: RO
Content-Length: 58

This case was approved in the ARC meeting today.

-Frank


