[cifs-protocol] RE: Help wanted - simple python or perl script for schema conversion

[Richard Guthrie]

All,

Thanks for the feedback, let me make sure I have correctly captured the issues you encountered with the schema doc:

1.)     Attribute msDS-ResultantPSO is missing the attributeId field
2.)     You observe the presence of two attribute values for a single valued field (Example is system flags in the included patch file).

Let me know if this is correct or if I missed anything and we can take a look at both of these.

Richard Guthrie
Open Protocols Support Team
Support Escalation Engineer, US-CSS DSC PROTOCOL TEAM
Tel: +1 (469) 775-7794
E-mail: rguthrie at microsoft.com




-----Original Message-----
From: Andrew Bartlett [mailto:abartlet at samba.org]
Sent: Wednesday, December 10, 2008 7:33 PM
To: S P
Cc: Richard Guthrie; pfif at tridgell.net; cifs-protocol at samba.org
Subject: Re: Help wanted - simple python or perl script for schema conversion

On Wed, 2008-12-10 at 21:46 +0530, S P wrote:
> On Wed, Dec 10, 2008 at 12:37 PM, Andrew Bartlett <abartlet at samba.org> wrote:
> > On Tue, 2008-12-09 at 20:20 +0530, S P wrote:
> >> On Tue, Dec 9, 2008 at 6:52 PM, S P <sree314 at gmail.com> wrote:
> >>
> >> This version handles CRLF/LF using python's universal newlines
> >> support, so it should work on DOS-style files as well.
> >>
> >> I've also rewritten the LDIF line-unfolding code, and it matches
> >> the spec more closely now.
> >>
> >> Let me know if it works now.
> >
> > I'm still having a few odd problems, but I've got permission to
> > publish the text-format schema, so you can find that in a current
> > GIT checkout in source4/setup/ad-schema.
>
> Thanks! I checked out the files you were using, and they are indeed
> CR-LF files which should've worked with the last program I sent you.
> However, they also contain three types of errors that will prevent it
> from working.

I've CC'ed Richard Guthrie from Microsoft, and the mailing lists we use to co-ordinate fixing this documentation.  I hope he can you can work to get this file into correct shape.

Richard, can you look into these problems with the 'text-file' schema you sent me a few months back:

> Firstly, there are spurious blank lines -- creating a separate record
> (this gives the ldapDisplayName not found error). I fixed these.
>
> The second error is that attribute msDS-ResultantPSO is missing the
> attributeId field
>
> The third error is the presence of two attr: values for a single
> valued field, systemFlags, with differing values.
>
> I've fixed the latter two of these errors by comparing to the docs, a
> patch is attached. I've not compared all of the MS files with the ones
> I extracted from the docs, though.
>
> Applying the patch is necessary to make the code I sent you run,
> though I've added some debug statements now as well. I'll post the
> code to the list, but should I post the current patch to fix the text
> files as well?

I don't think posting the patch should be a problem.  I've re-attached it here so Richard can see the problem.

> > What I now need is a clear copyright statement from you, including
> > any statement from minschema (if you copied code from that),
>
> Neither minschema.js nor minschema.py contains any statement :(
> However my code clearly contains code copied from minschema.py. I'll
> note this in the message to the mailing list. Will follow other
> instructions as well.

I'll get that fixed.

Thanks,

Andrew Bartlett

--
Andrew Bartlett
http://samba.org/~abartlet/
Authentication Developer, Samba Team           http://samba.org
Samba Developer, Red Hat Inc.