Bug 306321 – Installing sys-apps/smartmontools on g/fbsd

• Status: RESOLVED OBSOLETE
• Product: Gentoo/Alt

[Description Dmitri Bogomolov 2010-02-22 10:15:19 UTC]

There is a problems with building sys-apps/smartmontools
 1. on freebsd no need for virtual/mailx dependency:

equery belongs /usr/bin/mailx 
 * Searching for /usr/bin/mailx ... 
sys-freebsd/freebsd-ubin-6.2-r2 (/usr/bin/mailx)
 
 2. the file os_freebsd.cpp contains extra { on line 275

 3. sys-apps/smartmontools-5.39 doesn't build on freebsd-6.2 for me

Reproducible: Always

[Comment 1 Dmitri Bogomolov 2010-02-22 10:20:59 UTC]

Created attachment 220675 [details, diff]
my ebuild diff

conditional dependency + epatch

[Comment 2 Dmitri Bogomolov 2010-02-22 10:21:52 UTC]

Created attachment 220677 [details, diff]
eliminate extra { in os_freebsd.cpp

[Comment 3 Diego Elio Pettenò (RETIRED) 2010-02-22 13:51:10 UTC]

You most likely want freebsd-ubin to provide mailx to begin with, instead, so that you don't have to start adding conditional dependencies all over the place.

[Comment 4 Dmitri Bogomolov 2010-02-22 15:00:24 UTC]

(In reply to comment #3)
> You most likely want freebsd-ubin to provide mailx to begin with, instead, so
> that you don't have to start adding conditional dependencies all over the
> place.
> 

did you mean it must be
RDEPEND="!minimal? ( || ( sys-freebsd/freebsd-ubin virtual/mailx ) )
?

[Comment 5 Johan Hattne 2010-02-22 15:42:16 UTC]

(In reply to comment #4)
> (In reply to comment #3)
> > You most likely want freebsd-ubin to provide mailx to begin with, instead, so
> > that you don't have to start adding conditional dependencies all over the
> > place.
> > 
> 
> did you mean it must be
> RDEPEND="!minimal? ( || ( sys-freebsd/freebsd-ubin virtual/mailx ) )
> ?

What about PROVIDE="virtual/mailx" in sys-freebsd/freebsd-ubin?

[Comment 6 Johan Hattne 2010-02-22 15:43:34 UTC]

*** Bug 303293 has been marked as a duplicate of this bug. ***

[Comment 7 Dmitri Bogomolov 2010-02-22 15:48:04 UTC]

(In reply to comment #5)
> (In reply to comment #4)
> > (In reply to comment #3)
> > > You most likely want freebsd-ubin to provide mailx to begin with, instead, so
> > > that you don't have to start adding conditional dependencies all over the
> > > place.
> > > 
> > 
> > did you mean it must be
> > RDEPEND="!minimal? ( || ( sys-freebsd/freebsd-ubin virtual/mailx ) )
> > ?
> 
> What about PROVIDE="virtual/mailx" in sys-freebsd/freebsd-ubin?
> 

WOW. It should be another bug, I think. This is just about [trivial?] patching to get sys-apps/smartmontools work.

btw I live in obsolete world of g/fbsd-6.2 primarily :)

[Comment 8 Johan Hattne 2010-02-22 17:19:28 UTC]

(In reply to comment #7)
> (In reply to comment #5)
> > What about PROVIDE="virtual/mailx" in sys-freebsd/freebsd-ubin?
> 
> WOW. It should be another bug, I think. This is just about [trivial?] patching
> to get sys-apps/smartmontools work.

I marked 303293 as a duplicate of this one.

[Comment 9 Dmitri Bogomolov 2011-02-08 23:29:32 UTC]

Newer =sys-apps/smartmontools-5.40 don't have typos and play well with gmake :)
But it probably depends on sys-freebsd/freebsd-lib[usb].

[Comment 10 Dmitri Bogomolov 2011-02-08 23:40:42 UTC]

Created attachment 261889 [details, diff]
Conditional dependencies for =sys-apps/smartmontools-5.40

[Comment 11 Dmitri Bogomolov 2011-07-29 13:42:29 UTC]

Anyone, please, apply patch and throw away virtual/mailx or ~x86-fbsd.

[Comment 12 Dmitri Bogomolov 2011-07-29 13:56:14 UTC]

Created attachment 281449 [details, diff]
smartmontools-5.41.ebuild.diff

[Comment 13 Dmitri Bogomolov 2011-07-29 13:58:03 UTC]

Created attachment 281451 [details, diff]
smartmontools-5.41.ebuild.diff

[Comment 14 Dmitri Bogomolov 2011-12-11 18:59:11 UTC]

Created attachment 295483 [details, diff]
smartmontools-5.42.ebuild.diff

[Comment 15 Dmitri Bogomolov 2014-04-30 09:32:40 UTC]

The actual sys-apps/smartmontools-6.2 doesn't need patches and have ~x86-fbsd keyword already.