Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#46039 closed submission (fixed)

New port: libcdio-paranoia

Reported by: pgf Owned by: g5pw (Aljaž Srebrnič)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: Ionic (Mihai Moldovan)
Port: libcdio-paranoia

Description

Attached the Portfile for a new port: libcdio-paranoia version 10.2+0.93+1.
This once was part of the libcdio library, but was split since libcdio version 0.90.
The libcdio-paranoia library is used for audio extraction from a CD-DA in a jitter- and error-tolerant way.
I've updated the cmus player Portfile in order to support audio CD playback through libcdio/libcdio-paranoia. I tested it on my platform and everything is working fine. I'll submit the patch file in a different ticket.

Attachments (1)

Portfile (1.5 KB) - added by pgf 9 years ago.

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by pgf

Attachment: Portfile added

comment:1 Changed 9 years ago by pgf

cmus music player Portfile patched to support audio CD playback through libcdio-paranoia, ticket:46045.

comment:2 Changed 9 years ago by mf2k (Frank Schima)

Version: 2.3.3

comment:3 Changed 9 years ago by g5pw (Aljaž Srebrnič)

Owner: changed from macports-tickets@… to g5pw@…
Status: newassigned

Hello, thank you for the patch! I have some comments before I can merge it, however:

  • You can't have two URLs as homepage, you need to choose one.
  • As primary category, maybe audio would be more appropriate.
  • I'm pretty sure master_sites, distname and worksrcdir are set up correctly by the github PortGroup

comment:4 in reply to:  3 Changed 9 years ago by pgf

Sorry, for the late answer.

  • You can't have two URLs as homepage, you need to choose one.

OK, you can remove the github link.

  • As primary category, maybe audio would be more appropriate.

OK, I agree.

  • I'm pretty sure master_sites, distname and worksrcdir are set up correctly by the github PortGroup

Feel free to modify and experiment with the Portfile as you like.

comment:5 Changed 9 years ago by g5pw (Aljaž Srebrnič)

Resolution: fixed
Status: assignedclosed

Commited in r132174, thanks. Could you take maintainership since you submitted the port?

comment:6 Changed 9 years ago by Ionic (Mihai Moldovan)

Cc: ionic@… added

One stupid question, though. Why is it using a github repo, when the tarball is also available on, for instance, https://ftp.gnu.org/gnu/libcdio/ -- which is arguably more trustworthy than a random github repository?

comment:7 in reply to:  6 Changed 9 years ago by pgf

Replying to ionic@…:

One stupid question, though. Why is it using a github repo, when the tarball is also available on, for instance, https://ftp.gnu.org/gnu/libcdio/ -- which is arguably more trustworthy than a random github repository?

I was not aware of the existance of the tarball on the GNU FTP server. Feel free to change it if you believe that the GNU FTP server is a safer option.

Note: See TracTickets for help on using tickets.