Hi Rick,
My cut-and-paste error -
I agree totally that we do NOT want to use mofpretty when changing
existing classes - it would cause an 80% change rate in OutputTray.
Therefore, when I changed 'output tray' to 'PrintOutputTray' I tried to
stay within existing lines (as long as they didn't go past 79 columns).
Cheers,
- Ira
On Jan 22, 2008 3:02 PM, <Richard_Landau at dell.com> wrote:
> Thank you for the new MOFs. However, I don't think we want to run
> mofpretty on any existing classes, only on new classes. Here's the
> situation:
>> - For PrintInputTray and PrintOutputTray, from the 2.17 version of the
> MOF file to the mofinput version, 30% to 40% of all lines contain some
> difference (by actual count). This requires much, *much* editing.
>> - Many of the differences are editorial in nature, e.g., changing "input
> tray" to "PrintInputTray," subunit to Printer, and the like. These
> require red-blue changes, but are reasonable; that's what we agreed to
> do.
>> - Some of them are property name changes, e.g., from
> CriticalAlertsStatus to CriticalAlertsPresent, which more accurately
> reflects the question to which the boolean value is the answer.
>> - Some of them are ine-wrap changes, e.g., a word or an opening or
> closing bracket being on one line or the following line. These I will
> try to ignore. And it's just these that mofpretty generates many of.
>> Now for the sob story. I spent many hours red-blue editing the MOF for
> CIM_Printer only to discover that the course I had chosen permitted
> *only* easy red-blue editing *or* proper formatting. Pick one; not
> both. Gack. Five hours down the drain. And this was after ignoring
> all formatting changes and editing only whole lines or clauses. I am
> going to inquire if I can replace entire property definitions, with
> appropriate assurances that nothing funny is going on, given the number
> of changes.
>> I apologize to the group for having wandered down the wrong path so long
> and now having to retrace my steps. This will cost us at least one week
> in submitting the CRs.
>> Continuing to work on it....
>> rick
>>> -----Original Message-----
> From: owner-wims at pwg.org [mailto:owner-wims at pwg.org] On Behalf Of Ira
> McDonald
> Sent: Tuesday, January 22, 2008 12:52
> To: wims at pwg.org; Ira McDonald
> Subject: WIMS> CIM - Posted InputTray and OutputTray (22 Jan 2008)
>> Hi folks, Tuesday (22 January 2008)
>> I've just posted two CIM printing classes to:
>>ftp://ftp.pwg.org/pub/pwg/wims/cim/mofinput/ira-20080122.zip>> rel_0122.txt
> - this release note
> CIM_PrintInputTray.mof
> - minor NEW version - prtInputEntry
> CIM_PrintOutputTray.mof
> - major NEW version - prtOutputEntry
>> Note - Unfortunately, the changes to OutputTray are extensive - because
> right after it was TC approved, CIM Core made us: replace the term
> subunit with component everywhere; change parent to PrinterElement;
> rewrite all of the status properties (removing "Status:"); and
> capitalize formal class names. And Rick and I found our 'AtLeastOne'
> behavior bug (now fixed). The changes to InputTray are to change parent
> to PrinterElement and to capitalize formal class names.
>> Rick - please run mofpretty on these classes and post result to:
>>ftp://ftp.pwg.org/pub/pwg/wims/cim/mofoutput/>> Cheers,
> - Ira
>>> --
> Ira McDonald (Musician / Software Architect) Chair - Linux Foundation
> Open Printing WG Blue Roof Music/High North Inc
> email: blueroofmusic at gmail.com> winter:
> 579 Park Place Saline, MI 48176
> 734-944-0094
> summer:
> PO Box 221 Grand Marais, MI 49839
> 906-494-2434
> ------------------------------------------------------------------------
>> Change Log:
>> (CIM_PrintInputTray)
> - Changed Version to "2.18"
>> - Added note about association class CIM_PrinterComponent in class
> header (per John Crandall and CIM v2.17)
>> - Added MappingStrings to prtInputEntry to class header (per CIM v2.17)
>> - Changed CIM_LogicalElement to CIM_PrinterElement in class header
> (per John Crandall and CIM v2.17)
>> - Capitalized formal class names, 'input tray' to 'PrintInputTray',
> in dozens of places (per John Crandall and CIM v2.17)
>> - Deleted redundant InstanceID and SNMPRowId inherited from
> CIM_PrinterElement (per John Crandall and CIM v2.17)
>> - Added ModelCorrespondence to LocalizedDescription (per John Crandall)
>>> (CIM_PrintInputTray)
> - Changed Version to "2.18"
>> - Changed term "Subunit" to "Component" (per John Crandall)
>> - Added note about association class CIM_PrinterComponent in class
> header (per John Crandall and CIM v2.17)
>> - Added MappingStrings to prtInputEntry to class header (per CIM v2.17)
>> - Changed CIM_LogicalElement to CIM_PrinterElement in class header
> (per John Crandall and CIM v2.17)
>> - Capitalized formal class names, 'output tray' to 'PrintOutputTray',
> in dozens of places (per John Crandall and CIM v2.17)
>> - Deleted redundant InstanceID and SNMPRowId inherited from
> CIM_PrinterElement (per John Crandall and CIM v2.17)
>> - Added IsDefault (per John Crandall and CIM v2.17)
>> - Added ModelCorrespondence to LocalizedDescription (per John Crandall)
>> - Replaced MaxCapacityBasis with correct text from PrintInputTray
> (per John Crandall and CIM v2.17)
>> - Revised RemainingCapacityBasis to correct AtLeastOne bug
> (per John Crandall and CIM v2.17)
>> - Replaced AvailabilityStatus, NonCriticalAlertsPresent, and
> CriticalAlertsPresent with correct text from PrintInputTray
> (per John Crandall and CIM v2.17)
>> - Added OtherAvailabilityStatus (per John Crandall and CIM v2.17)
>> - Deleted OfflineStatus (per John Crandall and CIM v2.17)
>> ------------------------------------------------------------------------
>
--
Ira McDonald (Musician / Software Architect)
Chair - Linux Foundation Open Printing WG
Blue Roof Music/High North Inc
email: blueroofmusic at gmail.com
winter:
579 Park Place Saline, MI 48176
734-944-0094
summer:
PO Box 221 Grand Marais, MI 49839
906-494-2434