{{{
os.chmod(ldif_file, 0o777)
}}}
I think it should be 0644

note that if you are running this as non-root, you will not be able to chown

revision
0001-Ticket-47848-Add-new-function-to-create-ldif-files.patch

Replying to [comment:3 rmeggins]:

{{{
os.chmod(ldif_file, 0o777)
}}}
I think it should be 0644

note that if you are running this as non-root, you will not be able to chown

Yeah nice catch. New patch attached.

{{{
os.chmod(ldif_file, 0o644)
}}}
Do you really need the "o" between the "0" and the "644"? That looks wrong to me.

Replying to [comment:5 rmeggins]:

{{{
os.chmod(ldif_file, 0o644)
}}}
Do you really need the "o" between the "0" and the "644"? That looks wrong to me.

It's for python 3 compliance, and python 2.7 handles it correctly.

To ssh://git.fedorahosted.org/git/389/lib389.git
6847117..49f1155 master -> master
commit 49f11558c761a419540b5d34f840055132026505
Author: Mark Reynolds mreynolds@redhat.com
Date: Thu Aug 13 16:12:23 2015 -0400

Updated 389-ds-base testsuite

To ssh://git.fedorahosted.org/git/389/ds.git
70ccd9b..dfb1c31 master -> master
commit dfb1c31

c16382c..7ea4487 389-ds-base-1.3.4 -> 389-ds-base-1.3.4
commit 7ea4487

20f6bc6..dc22924 389-ds-base-1.3.3 -> 389-ds-base-1.3.3
commit dc22924

Milestone lib389 1.0 deleted

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/389ds/389-ds-base/issues/1579

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.