[New port] audio/pianod2: A free, multi-source, network-controlled music player daemon

[jhixson]

Summary

[New port] audio/pianod2: A free, multi-source, network-controlled music player daemon

Test Plan

Tested with poudriere for 11.2 and 12.0

[mat]

audio/pianod2/Makefile

16

USES=pkgconfig

36

Wrong place in the Makefile. See Chapter 15. Order of Variables in Port Makefiles.

39

This is already in the default value and is not needed.

52

Why +=?

[jhixson]

audio/pianod2/Makefile

36

I did RTFM here. I am no officially confused. The documentation says that standard bsd,port.mk variables come after USE_* variables. GNU_CONFIGURE is in bsd.port.mk. So where should this go if in fact not after USE_* variables?

52

Oops, cut & paste. Will fix.

[jhixson]

audio/pianod2: port fixes

• Add pkgconfig to USES
• Fix typo
• Fix assignment to make arguments

[mat]

audio/pianod2/Makefile

36

It should go in the Standard bsd.port.mk Variables section.
Maybe I should have tagged the SHEBANG_FILES and ksh_CMD lines instead, those should go in the USES and USE_x section.
I will let you tell me why as an exercise :-)

[jhixson]

Please help me understand this ;-) Here is Chapter 15 of the handbook:

Chapter 15. Order of Variables in Port Makefiles
Table of Contents

15.1. PORTNAME Block
15.2. PATCHFILES Block
15.3. MAINTAINER Block
15.4. LICENSE Block
15.5. Generic BROKEN/IGNORE/DEPRECATED Messages
15.6. The Dependencies Block
15.7. Flavors
15.8. USES and USE_x
15.9. Standard bsd.port.mk Variables
15.10. Options and Helpers
15.11. The Rest of the Variables
15.12. The Targets

I assume the ordering is sequential here. 15.8 comes before 15.9. 15.8 is the USES and USE_* variables. 15.9 is the standard bsd.port.mk variables. GNU_CONFIGURE is in the bsd.port.mk file, so I consider that a "standard bsd.port.mk" variable. What am I missing/misunderstanding here?

[mat]

Well, thie thing is, you have to actually read the sections, looking at the titles is not enough, they are not long. For example, USES and USE_x says:

Start this section with defining USES, and then possible USE_x.
Keep related variables close together. For example, if using USE_GITHUB, always put the GH_* variables right after it.

The second line means that any variable related to USES or USE have to get in this section.

Do you get where I am going with this?

[jhixson]

The only example given is for github, and yes, I did read it ;-) What exactly would GNU_CONFIGURE fall under? There isn't a USE_GNU or any other such variable. I *still* don't know where this is supposed to go. Maybe the handbook should be more clear. How about passing on the knowledge that I am missing? I really don't know where this is supposed to go and the handbook does not make it clear at all.

[mat]

Yes, there is one example, from it, you can infer that the SHEBANG_FILES and ksh_CMD are related to USES=shebangfix. and should go in the USES/USE_* section.

[jhixson]

I have no idea where GNU_CONFIGURE goes. I've asked for it to be pointed out to where it goes. Since I can't get any help on where to put it, I'm just going to close this.

[araujo]

Please, reopen the review, when I'm back from holidays I will help you.

[jhixson]

Sooooooooo. Is anybody going to help me fix this or approve this? ;-) I am still waiting.

[araujo]

In D19042#422183, @jhixson wrote:

Sooooooooo. Is anybody going to help me fix this or approve this? ;-) I am still waiting.

Hey,

I will do it tonight (Taiwan)!

Best,