> On Jan. 2, 2017, 5:50 p.m., Don Bosco Durai wrote: > > Thanks, I was now able to apply the patch. I have not run it though. > > Pradeep, do you want to give a try at this? Thanks > > Pradeep Agrawal wrote: > =>Tested patch and its working fine for below mentioned files : > kms/scripts/db_setup.py > kms/scripts/dba_script.py > security-admin/scripts/db_setup.py > security-admin/scripts/dba_script.py > > =>Below mentioned files are not working and will not work as setup.sh of > respective module need to be executed first. > security-admin/scripts/changepasswordutil.py > tagsync/scripts/updatetagadminpassword.py > security-admin/scripts/restrict_permissions.py > > Zsombor Gegesy wrote: > What's your suggestions? Should I drop the changes from that 3 files? Or > can it be merged?
Yes, let's drop those changes for now. - Pradeep ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54893/#review160333 ----------------------------------------------------------- On Dec. 28, 2016, 9:37 p.m., Zsombor Gegesy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54893/ > ----------------------------------------------------------- > > (Updated Dec. 28, 2016, 9:37 p.m.) > > > Review request for ranger, Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, > and Velmurugan Periasamy. > > > Bugs: RANGER-1262 > https://issues.apache.org/jira/browse/RANGER-1262 > > > Repository: ranger > > > Description > ------- > > Setting up database on MacOS is not working, because the db > creation/migration script contains Linux and Windows specific code > > > Diffs > ----- > > kms/scripts/db_setup.py f1342fd > kms/scripts/dba_script.py 4366743 > security-admin/scripts/changepasswordutil.py 3086962 > security-admin/scripts/db_setup.py 7cd1ef0 > security-admin/scripts/dba_script.py 1b29c75 > security-admin/scripts/restrict_permissions.py a4998be > tagsync/scripts/updatetagadminpassword.py 2c89e83 > > Diff: https://reviews.apache.org/r/54893/diff/ > > > Testing > ------- > > > Thanks, > > Zsombor Gegesy > >
