Re: error building file_contexts

[Daniel J Walsh <dwalsh@redhat.com>]

Eric Paris wrote:

>>(...)
>>-/home/rsyslog
>>-/emul/lib(64)?/*$
>>-/home/rsyslog
>>-/emul/lib(64)?/.*\.so(\.[^/]*)*/*$
>>-/home/rsyslog
>>-/emul/lib(64)?/ld-[^/]*\.so(\.[^/]*)*/*$
>>-/home/rsyslog
>>-/usr/sbin/amavisd(.*)?/*$
>>-/home/rsyslog
>>Traceback (most recent call last):
>>  File "/usr/sbin/genhomedircon", line 475, in ?
>>    oldgenhomedircon(cmds[0], cmds[1])
>>  File "/usr/sbin/genhomedircon", line 190, in oldgenhomedircon
>>    if re.search(regex, potential, 0):
>>  File "/usr/lib/python2.3/sre.py", line 137, in search
>>    return _compile(pattern, flags).search(string)
>>  File "/usr/lib/python2.3/sre.py", line 230, in _compile
>>    raise error, v # invalid expression
>>sre_constants.error: nothing to repeat
>>make: *** [file_contexts/file_contexts] Error 1
>>
>>Any ideas?
>>
>>Thanks
>>Antoine
>>
>>
>>For reference, here is the updated code snippet:
>>(...)
>>	for potential in potential_prefixes.keys():
>>                addme = 1
>>                for regex in prefix_regex:
>>                        #match a trailing (/*)? which is actually a bug
>>in rpc_pipefs
>>                        regex = re.sub("\(/\*\)\?$", "", regex)
>>                        #match a trailing .+
>>                        regex = re.sub("\.+$", "", regex)
>>                        #match a trailing .*
>>                        regex = re.sub("\.\*$", "", regex)
>>                        #strip a (/.*)? which matches anything trailing
>>to a /*$ which matches trailing /'s
>>                        regex = re.sub("\(\/\.\*\)\?", "", regex)
>>                        regex = regex + "/*$"
>>                        sys.stderr.write("-%s\n" % regex)
>>                        sys.stderr.write("-%s\n" % potential)
>>                        sys.stderr.flush()
>>                        if re.search(regex, potential, 0):
>>                                addme = 0
>>
>>                if addme == 1:
>>                        if not prefixes.has_key(potential):
>>                                prefixes[potential] = ""
>>(...)
>>    
>>
>
>
>So I simplified the problem.  Maybe someone with more regex or python
>foo can help us out.  Python refuses to use (.*)? as a valid regular
>expression.  Seems like we want to say match anything 0 or more times.
>Why is this invalid and what would be valid?
>
>  
>
>>>>import re
>>>>test = re.sub("(.*)?", "", "hello")
>>>>        
>>>>
>Traceback (most recent call last):
>  File "<stdin>", line 1, in ?
>  File "/usr/lib/python2.3/sre.py", line 143, in sub
>    return _compile(pattern, 0).sub(repl, string, count)
>  File "/usr/lib/python2.3/sre.py", line 230, in _compile
>    raise error, v # invalid expression
>sre_constants.error: nothing to repeat
>
>
>
>--
>This message was distributed to subscribers of the selinux mailing list.
>If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
>the words "unsubscribe selinux" without quotes as the message.
>  
>
Why not just .*

test=re.sub(".*","","hello")



-- 



--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.