#160: [PATCH] Edit: ebuild syntax definition
-------------------------------------------------------------------------------+
  Reporter:  enrico.weig...@zaphod.local, metux IT service <weig...@metux.de>  
|       Owner:  metux         
      Type:  enhancement                                                       
|      Status:  accepted      
  Priority:  major                                                             
|   Milestone:  4.7           
 Component:  mcedit                                                            
|     Version:  4.6.1         
Resolution:                                                                    
|    Keywords:  review, ebuild
  Blocking:                                                                    
|   Blockedby:                
-------------------------------------------------------------------------------+
Changes (by slyfox):

  * keywords:  review => review, ebuild


Comment:

 .ebuilds are usual shell scripts

 Ebuild syntax file looks _OUTDATED_ in respect to shell.syntax with:

 shell out-of-sync issues:
 * some control words are hot highlighted: 'if', 'then', 'else', etc. (have
 a look at 'diff syntax/ebuild.syntax syntax/shell.syntax')
 * some literals are not highlighted:
     * single quoted ones: """sed -e -s's/hello/world/g'"""
     * global variables: """AUDIO="$(get_audio)"""

 * some global vars are not highlighted properly: SCR_URI, SLOT
 * some ebuild keywords are not highlighted at all: 'enewgroup'

 All the rest looks OK, so patch could be merged. I could add some patches
 to fix above issues this weekend :]

-- 
Ticket URL: <www.midnight-commander.org/ticket/160#comment:11>
Midnight Commander <www.midnight-commander.org>
Midnight Development Center
_______________________________________________
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel

Reply via email to