Opened 10 years ago

Closed 10 years ago

#46197 closed enhancement (fixed)

bladeRF: Improve version strings

Reported by: jon.szymaniak@… Owned by: michaelld (Michael Dickens)
Priority: Low Milestone:
Component: ports Version:
Keywords: Cc:
Port: bladeRF

Description

Currently, the bladeRF (libbladeRF and bladeRF-cli) build attempts to gather information from git, which it does not find during a Macports build. This results in version strings like this, which don't offer any additional information and may be concerning to users.

bladeRF> version

  bladeRF-cli version:        0.12.0-git-unknown
  libbladeRF version:         0.17.0-git-unknown

This "-git-unknown" suffix can be removed by adding the following to the port's configure.args-append:

-DVERSION_INFO_EXTRA=""

However, I think it would be very useful if we instead applied a meaningful MacPorts version annotation via -DVERSION_INFO_OVERRIDE.

I can say with great certainty that this would be helpful in situations where I've seen where folks end up confused because they've got libbladeRF installed in multiple places (one from MacPorts, another from a previous source build) and some PATH or dyld path confusion.

Therefore, I propose appending the following to configure.args-append.

-DVERSION_INFO_OVERRIDE="-MacPorts-${version}"

In this case, I intended ${version} to be the Port version. My syntax may be incorrect, but I suspect this gets the point across. The point is to have versions read back like this:

bladeRF> version

  bladeRF-cli version:        0.12.0-MacPorts-20141120
  libbladeRF version:         0.17.0-MacPorts-20141120

Note that there's no need to touch -DVERSION_INFO_EXTRA in this case, as -DVERSION_INFO_OVERRIDE takes priority.

For what it's worth, I've attempted to document this a bit here, as none of this is obvious otherwise.

Change History (4)

comment:1 Changed 10 years ago by jon.szymaniak@…

Cc: jon.szymaniak@… added

My apologies, I failed to CC the port maintainer. Given that I don't see a way to update the ticket, I'll touch base with him offline to note the creation of this ticket.

Update: Nevermind - Thank you larryv!

Last edited 10 years ago by jon.szymaniak@… (previous) (diff)

comment:2 Changed 10 years ago by larryv (Lawrence Velázquez)

Cc: jon.szymaniak@… removed
Owner: changed from macports-tickets@… to michaelld@…
Port: bladeRF added
Version: 2.3.3

This sounds like a pretty good idea.

comment:3 Changed 10 years ago by michaelld (Michael Dickens)

Excellent idea. I'll add that with the next upgrade tonight. Thanks for pointing this out; I had no idea.

comment:4 Changed 10 years ago by michaelld (Michael Dickens)

Resolution: fixed
Status: newclosed

Done in r129364.

Note: See TracTickets for help on using tickets.