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
Created attachment 220675 [details, diff] my ebuild diff conditional dependency + epatch
Created attachment 220677 [details, diff] eliminate extra { in os_freebsd.cpp
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.
(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 ) ) ?
(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?
*** Bug 303293 has been marked as a duplicate of this bug. ***
(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 :)
(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.
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].
Created attachment 261889 [details, diff] Conditional dependencies for =sys-apps/smartmontools-5.40
Anyone, please, apply patch and throw away virtual/mailx or ~x86-fbsd.
Created attachment 281449 [details, diff] smartmontools-5.41.ebuild.diff
Created attachment 281451 [details, diff] smartmontools-5.41.ebuild.diff
Created attachment 295483 [details, diff] smartmontools-5.42.ebuild.diff
The actual sys-apps/smartmontools-6.2 doesn't need patches and have ~x86-fbsd keyword already.