Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 729280 - app-emacs/ebuild-mode: add BDEPEND to new ebuilds
Summary: app-emacs/ebuild-mode: add BDEPEND to new ebuilds
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Emacs project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-06-23 09:27 UTC by Joonas Niilola
Modified: 2022-01-16 11:44 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Joonas Niilola gentoo-dev 2020-06-23 09:27:20 UTC
When using 'ebuild-mode-skeleton' to create a completely new ebuild from scratch, it leaves out BDEPEND variable. It already suggests EAPI-7 to be used, so I feel like BDEPEND should be there.

From what I could tell, simple patch like this was enough: 

diff -Naur ebuild-mode-1.50/ebuild-mode.el ebuild-mode-1.50.1/ebuild-mode.el
--- ebuild-mode-1.50/ebuild-mode.el	2020-03-08 00:36:27.000000000 +0200
+++ ebuild-mode-1.50.1/ebuild-mode.el	2020-06-23 12:18:45.096162213 +0300
@@ -555,7 +555,8 @@
   "\n"
   ;; dependencies
   "RDEPEND=\"\"\n"
-  "DEPEND=\"\$\{RDEPEND\}\"\n")
+  "DEPEND=\"\$\{RDEPEND\}\"\n"
+  "BDEPEND=\"\"\n")
 
 
 ;;; Keybindings.
Comment 1 Ulrich Müller gentoo-dev 2020-06-23 10:57:49 UTC
Omitting BDEPEND was deliberate, because (at least in my understanding) policy still allows listing all build-time dependencies in DEPEND. See the Council log where EAPI 7 features have been approved: https://projects.gentoo.org/council/meeting-logs/20171112.txt

[20:26:22] <mgorny> the relevant policy would be that developers are free to ignore BDEPEND and let people who care about cross fix it
[20:26:48] <mgorny> (like they don't have to be 100% prefix safe right now)
[20:26:54] <dilfridge> "fix it" = distribute things between BDEPEND and DEPEND
[20:27:00] <ulm> yep