[PATCH] Edit: ebuild syntax definition

[Enrico.Weigelt@…>]

Adding Gentoo eBuild syntax definition.

changeset:5471443eeacb91322fb7488bcd24b4eb6a609317

[anonymous]

Added by email2trac

[anonymous]

This message has 1 attachment(s)

[winnie]

I reassign this to you as you created the branch 160_syntax_ebuild which contains the attached fix. See b0243d50783ad78995909f354e25db204c2d8af7 and 3906a150e6cd400e9508309c947b3f235d776b72 for details.

[winnie]

Playing around with the default queries.. in order to test if a modified query works I need a dummy vote-user tag...

[winnie]

Adding my vote for testing purposes.. will remove it later

[winnie]

removed testing stuff

[Enrico Weigelt]

• MC Ticket System <tickets@…> schrieb:

Playing around with the default queries.. in order to test if a modified
query works I need a dummy vote-user tag...

hmm, can you set up an query which shows all with at least two
votes ?

cu

[winnie]

As I'm not familiar with ebuild at all I can't say anything to the syntaxfile.. from the layout and etc. it looks good but if this file makes sense with these keywords.. I don't know.

Can please someone who knows ebuild files review this ticket please? If there is only one vote in the next week please add a vote-winnie as I think this file will be correct and I doesn't want to stand in the way to get this new syntaxfile into mc.

[slavazanko]

I'm Red Hat man, and don't know is keywords valid or not too. But syntax-files not important for evolve development, IMHO.

Propose: syntax-related patches may apply without vote (or with one vote only).

[winnie]

No.. every patch should need two votes.. if there are special cases (like this patch) it might be okay if someone who knows the ebuild syntax well can apply it without getting two votes..

As this patch seems to be okay from the "looks good standpoint" feel free to commit w/o my vote.

[slyfox]

.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 :]

[winnie]

Hey Slyfox,

It would be cool if you could send us a patch which is based on the 160_syntax_ebuild branch. Then we could directly add your patch there.

As you say that this patch is not fully okay I set keyword to "rework" until this is fixed.

Thanks for your help.

[slyfox]

Done.

0001-syntax-ebuild.syntax-new-syntax-file-based-on-sh.sy.patch

feel free to git-apply it on top of 160_syntax_ebuild branch

[slavazanko]

patch 0001-syntax-ebuild.syntax-new-syntax-file-based-on-sh.sy.patch applied to '160_syntax_ebuild' branch. See changeset:3bc655a89b326f3e8128a3805a06f7bf73caa467

Please, review someone who familiar with Gentoo :)

feel free to git-apply it on top of 160_syntax_ebuild branch

As I see, git-am better utility for applying patches like your patch :)

[slavazanko]

Branch '160_syntax_ebuild' merged to master.

Branch '160_syntax_ebuild' deleted.

[winnie]

Closing