Commit786ccf0

authored

Merge pull request#501 from postgrespro/enable_nls_check

Add checking enable-nls option in configure

2 parents7591667 +f544da1 commit786ccf0Copy full SHA for 786ccf0

File tree

+21

-6

lines changed

+21

-6

lines changed

Lines changed: 11 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -101,6 +101,16 @@ def is_enterprise():`
`101`	`101`	`else:`
`102`	`102`	`returnFalse`
`103`	`103`
	`104`	`+defis_nls_enabled():`
	`105`	`+cmd= [os.environ['PG_CONFIG'],'--configure']`
	`106`	`+`
	`107`	`+p=subprocess.Popen(`
	`108`	`+cmd,`
	`109`	`+stdout=subprocess.PIPE,`
	`110`	`+stderr=subprocess.PIPE`
	`111`	`+ )`
	`112`	`+returnb'enable-nls'inp.communicate()[0]`
	`113`	`+`
`104`	`114`
`105`	`115`	`classProbackupException(Exception):`
`106`	`116`	`def__init__(self,message,cmd):`
`@@ -147,6 +157,7 @@ def slow_start(self, replica=False):`
`147`	`157`	`classProbackupTest(object):`
`148`	`158`	`# Class attributes`
`149`	`159`	`enterprise=is_enterprise()`
	`160`	`+enable_nls=is_nls_enabled()`
`150`	`161`
`151`	`162`	`def__init__(self,args,*kwargs):`
`152`	`163`	`super(ProbackupTest,self).__init__(args,*kwargs)`

Lines changed: 10 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -231,9 +231,13 @@ def test_options_5(self):`
`231`	`231`	`# @unittest.skip("skip")`
`232`	`232`	`deftest_help_6(self):`
`233`	`233`	`"""help options"""`
`234`		`-self.test_env['LC_ALL']='ru_RU.utf-8'`
`235`		`-withopen(os.path.join(self.dir_path,"expected/option_help_ru.out"),"rb")ashelp_out:`
`236`		`-self.assertEqual(`
`237`		`-self.run_pb(["--help"]),`
`238`		`-help_out.read().decode("utf-8")`
`239`		`- )`
	`234`	`+ifProbackupTest.enable_nls:`
	`235`	`+self.test_env['LC_ALL']='ru_RU.utf-8'`
	`236`	`+withopen(os.path.join(self.dir_path,"expected/option_help_ru.out"),"rb")ashelp_out:`
	`237`	`+self.assertEqual(`
	`238`	`+self.run_pb(["--help"]),`
	`239`	`+help_out.read().decode("utf-8")`
	`240`	`+ )`
	`241`	`+else:`
	`242`	`+returnunittest.skip(`
	`243`	`+'You need configure PostgreSQL with --enabled-nls option for this test')`

Comments

(0)