Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 379169

Summary: app-vim/gentoo-syntax: new ebuild: swap DEPEND and RDEPEND
Product: Gentoo Linux Reporter: Sébastien Wilmet <sebastien.wilmet>
Component: [OLD] DevelopmentAssignee: Ben de Groot (RETIRED) <yngwin>
Status: RESOLVED FIXED    
Severity: trivial CC: darkside, idl0r, vim
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: 0001-Swap-rdepend-and-depend-in-newebuild.vim.patch

Description Sébastien Wilmet 2011-08-14 14:36:26 UTC
When we create a new ebuild file with vim, we get this at the end:

> DEPEND=""
> RDEPEND="${DEPEND}"

It must be replaced by:

> RDEPEND=""
> DEPEND="${RDEPEND}"

The bug is located at /usr/share/vim/vimfiles/plugin/newebuild.vim (line 155).

How to reproduce the bug:
$ cd; vim new.ebuild
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2011-08-15 02:02:59 UTC
You forgot to reason why these should be swapped. I could much more reasonably argue that neither should turn up in an empty ebuild template as neither is a required variable.
Comment 2 Sébastien Wilmet 2011-08-15 11:33:58 UTC
I'm not an expert at writing ebuilds. I thought that all ebuilds use the second form, which is more logical for me. Or, for more complex needs, a "common_depend" variable can be used. After grepping a little in the portage tree, actually it seems that a lot of ebuilds use the first form.

As a beginner, the first form was disturbing, so I reported this bug.
Comment 3 Christian Ruppert (idl0r) gentoo-dev 2012-01-03 01:21:38 UTC
Swapping those variables makes no sense to me as it always depends on the package.
I'd say WONTFIX.
Comment 4 Tomáš Chvátal (RETIRED) gentoo-dev 2012-03-16 06:15:13 UTC
Created attachment 305579 [details, diff]
0001-Swap-rdepend-and-depend-in-newebuild.vim.patch

I have to agree with the user.

When I looked on the tree most new stuff have these two variables swapped if they are filled.

Also it make sense wrt adding build only deps like doxygen and others.

And fwiw if you get them by default you can dd them quite easily, but most apps have deps anyway.
Comment 5 Patrice Clement gentoo-dev 2017-03-02 00:01:13 UTC
It was dealt with ages ago. No longer relevant. Closing.