[PATCH] Add support for MS Catalog files

Andrew Bartlett abartlet at samba.org
Tue Aug 7 21:37:21 UTC 2018


On Tue, 2018-08-07 at 12:21 -0700, Jeremy Allison via samba-technical
wrote:
> On Tue, Aug 07, 2018 at 11:40:39AM +0200, Andreas Schneider wrote:
> > 
> > Updated patchset attached.
> > 
> > 
> > Thanks for the review!
> 
> LGTM. Really nice, clean code - thanks !
> 
> Reviewed-by: Jeremy Allison <jra at samba.org>
> 
> However, it adds a dependency on the package libtasn1-bin
> for the ans1Parser program.
> 
> You'll need to get root at sn-devel to install that,
> or change the patch to gate the compilation of this binary on
> availability of libtasn1-bin, as currently it won't
> configure without it.

Also the package list used to build the docker image needs to be
updated:
http://git.catalyst.net.nz/gitweb?p=samba-cloud-autobuild.git;a=blob;f=
package-lists/ubuntu-1404
http://git.catalyst.net.nz/gitweb?p=samba-cloud-autobuild.git;a=blob;f=
package-lists/ubuntu

Then the docker container needs to be rebuilt (someone just has to re-
run an ansible script), and the .travis-ci.yml file needs to be
updated.

Not the end of the world, just needs to be kept in sync for our sanity.
 Patches to bring this list into master and to automatically populate
all the different outputs (there are also packer scripts) would be most
welcome.

I've pre-emptivly done the above (except for the root stuff, naturally)
and the .travis-ci.yml, please just include that in the patch. 

Andrew Bartlett
-- 
Andrew Bartlett
https://samba.org/~abartlet/
Authentication Developer, Samba Team         https://samba.org
Samba Development and Support, Catalyst IT   
https://catalyst.net.nz/services/samba







More information about the samba-technical mailing list