Re: [CEDET-devel] python-mode support

[Eric L. <er...@si...>]

Hi,

I think it makes sense to support a wide array of modes.  I'm not that familiar with the question you ask though.  Perhaps if you take out the require all together, and run the unit tests we can find out what it is used for.

Eric

Sent from Samsung tablet

Andrea Crotti <and...@gm...> wrote:

>Andrea Crotti <and...@gm...> writes:
>
>> Hi everyone, I realized some days ago that using semantic on python
>> code had a side-effect, I was no more using python-mode.el but the
>> default python.el.
>>
>> I found that the problem was given from wisent-python.el, and I think
>> I might have found a solution.
>>
>> Making the python library to load customizable and setting it to
>> python-mode seems to work (as the diff below).
>>
>> Only a few concerns about it:
>> - I can't find exactly for what python-mode is used for
>> - I'm not sure this is the right way to solve this "issue" and/or it might
>>   be generalized a bit more.
>>
>> === modified file 'semantic/wisent/wisent-python.el'
>> --- semantic/wisent/wisent-python.el	2011-01-20 21:12:43 +0000
>> +++ semantic/wisent/wisent-python.el	2011-02-26 19:48:31 +0000
>> @@ -44,9 +44,11 @@
>>  
>>  (require 'rx)
>>  
>> -;; Try to load python support, but fail silently since it is only used
>> -;; for optional functionality
>> -(require 'python nil t)
>> +;; try to load python support, failing silently if not found
>> +(defcustom semantic-python-mode-backend 'python
>> +  "Select the used python-mode library, for example 'python, 'python-mode")
>> +
>> +(require semantic-python-mode-backend nil t)
>>  
>>  (require 'semantic-wisent)
>>  (require 'wisent-python-wy)
>
>Noone interested in this?
>I'll open a blueprint on launchpad otherwise, maybe it's more visible...
>The solution I proposed here is surely not the best one but it works
>well enough for me.
>
>------------------------------------------------------------------------------
>Free Software Download: Index, Search & Analyze Logs and other IT data in 
>Real-Time with Splunk. Collect, index and harness all the fast moving IT data 
>generated by your applications, servers and devices whether physical, virtual
>or in the cloud. Deliver compliance at lower cost and gain new business 
>insights. http://p.sf.net/sfu/splunk-dev2dev 
>_______________________________________________
>Cedet-devel mailing list
>Ced...@li...
>https://lists.sourceforge.net/lists/listinfo/cedet-devel