#45679 closed update (fixed)
bsdowl: Update to v2.2.1
Reported by: | michipili (Michael Grünewald) | Owned by: | ryandesign (Ryan Carsten Schmidt) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | haspatch maintainer | Cc: | |
Port: | bsdowl |
Description
This is a simple update to v2.2.1.
References:
Attachments (2)
Change History (15)
Changed 10 years ago by michipili (Michael Grünewald)
Attachment: | patch-bsdowl.diff added |
---|
comment:1 Changed 10 years ago by mf2k (Frank Schima)
Keywords: | haspatch maintainer added |
---|---|
Port: | bsdowl added; devel/bsdowl removed |
Version: | 2.3.2 |
comment:3 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
The configure phase fails:
checking for gawk... gawk checking for grep that handles long lines and -e... /opt/local/bin/grep checking for a BSD-compatible install... /usr/bin/install -c checking whether ln -s works... yes checking for a thread-safe mkdir -p... /opt/local/bin/gmkdir -p checking for a sed that does not truncate output... /opt/local/bin/gsed checking for id that handles -g, -u and -n... id checking for gm... yes checking for bmake... no configure: error: *** BSD Make not found.
The program the bsdmake port provides is called bsdmake
not bmake
.
comment:5 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
michipili, can you comment on the configure failure I reported above? Do you have a solution?
comment:6 follow-up: 7 Changed 10 years ago by michipili (Michael Grünewald)
This is a missing dependency. The bsdowl
port should depend on devel/bmake
, for building and running. I can submit a patch in following days.
comment:7 follow-up: 8 Changed 10 years ago by michipili (Michael Grünewald)
This is is bigger problem than I thought.
First, as you ryandesign@… pointed it out, there is two BSD Make programs available in MacPorts:
- bsdmake
- bmake
These programs are very similar but distinct (at least, available versions are not compatible). The bsdowl
software is actually compatible with our MacPorts for bmake
but not bmake
.
Dependencies are not an issue here, rather the build.type
. Which value should I pick so that the make command used is bmake -r
instead of bsdmake
?
comment:8 Changed 10 years ago by larryv (Lawrence Velázquez)
Replying to michipili@…:
Dependencies are not an issue here, rather the
build.type
. Which value should I pick so that the make command used isbmake -r
instead ofbsdmake
?
There is no value of build.type
corresponding to bmake. You will probably need something like this (untested):
depends_build bin:bmake:bmake build.cmd bmake build.post_args -r
I’m just guessing that bmake accepts build targets as GNU Make does. You’ll probably have to tweak my example.
comment:9 follow-up: 10 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | ryandesign@… added |
---|---|
Summary: | devel/bsdowl: Update to v2.2.1 → bsdowl: Update to v2.2.1 |
If I install bmake, that satisfies the configure check. But given the existing "build.type bsdmake
" in the Portfile, the port then builds using bsdmake, which seems to work fine. Are we certain bsdowl really requires bmake? If bsdmake will work, we should make that happen, e.g. by bypassing the configure check for bmake as in attachment:bsdowl-2.2.1-ryandesign.diff.
Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
Attachment: | bsdowl-2.2.1-ryandesign.diff added |
---|
comment:10 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | ryandesign@… removed |
---|---|
Owner: | changed from macports-tickets@… to ryandesign@… |
Status: | new → assigned |
Replying to ryandesign@…:
Are we certain bsdowl really requires bmake?
I asked about this in the upstream issue tracker where the developer (who is also this ticket's reporter) explained that more bmake-specific functionality will be used in future versions of bsdowl, so it seems simplest for bsdowl in MacPorts to use bmake after all. We may want to add "build.type bmake
" functionality to MacPorts base at some point. For now, since bsdowl will be the first port in MacPorts to use bmake, we can do it manually as Larry suggested. I'll do that and commit the update.
comment:11 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:12 follow-up: 13 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
comment:13 Changed 10 years ago by michipili (Michael Grünewald)
Changed in r129879.
Thank you for your help!
Update to v2.1.1