Paths

Table of Contentst

Differential D12998

New port: multimedia/py-pretty_midi: Utility functions for handling MIDI data in a nice/intuitive way
ClosedPublic
Actions

Authored by yuri on Nov 8 2017, 5:04 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Oct 19, 4:06 AM

	Unknown Object (File)
	Sep 21 2025, 12:45 AM

	Unknown Object (File)
	Sep 18 2025, 9:46 PM

	Unknown Object (File)
	Sep 16 2025, 1:48 AM

	Unknown Object (File)
	Sep 14 2025, 10:21 AM

	Unknown Object (File)
	Aug 29 2025, 4:53 PM

	Unknown Object (File)
	Aug 25 2025, 5:16 AM

	Unknown Object (File)
	Aug 15 2025, 12:24 AM

Subscribers

Details

Reviewers

tcberner
adamw

Commits

rP454848: New port: multimedia/py-pretty_midi: Utility functions for handling MIDI data…

Summary

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=220752

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

yuri created this revision.Nov 8 2017, 5:04 AM

Herald added a subscriber: mat. · View Herald TranscriptNov 8 2017, 5:04 AM

tcberner added inline comments.Nov 8 2017, 6:48 AM

multimedia/py-pretty-midi/Makefile
15 ↗	(On Diff #34925)	py-mido is missing

tcberner added a parent revision: D12995: New port: multimedia/py-mido: MIDI Objects for Python.Nov 8 2017, 6:49 AM

Sorry, I didn't first create the dependency.

In D12998#269928, @yuri wrote:

Sorry, I didn't first create the dependency.

no worries :)

mat added inline comments.Nov 8 2017, 12:52 PM

multimedia/py-pretty-midi/Makefile
3 ↗	(On Diff #34925)	PORTNAME and directory name are not the same, one `_` and one `-`.

yuri marked 2 inline comments as done.Nov 12 2017, 4:54 AM

yuri added inline comments.Nov 12 2017, 8:49 AM

multimedia/py-pretty-midi/Makefile
3 ↗	(On Diff #34925)	GitHub directory has `-`, but pypi.python.org directory has `_`. I think we should go with the original version, which is GitHub.

yuri marked an inline comment as done.Nov 12 2017, 9:29 PM

tcberner added inline comments.Nov 17 2017, 10:48 PM

multimedia/py-pretty-midi/Makefile
3 ↗	(On Diff #34925)	Then you should also call the port 'pretty-midi' multimedia/py-pretty-midi -> py27-pretty_midi-0.2.8 just is not nice, if you ask me.

yuri marked an inline comment as done.Nov 23 2017, 6:42 AM

yuri added inline comments.

multimedia/py-pretty-midi/Makefile
3 ↗	(On Diff #34925)	So, the package is now called pretty_midi.

mat added inline comments.Nov 24 2017, 6:13 PM

multimedia/py-pretty-midi/Makefile
3 ↗	(On Diff #34925)	So the port directory should be with a `_`.

Renamed the port dir.

yuri marked 2 inline comments as done.Nov 24 2017, 7:52 PM

yuri retitled this revision from New port: multimedia/py-pretty-midi: Utility functions for handling MIDI data in a nice/intuitive way to New port: multimedia/py-pretty_midi: Utility functions for handling MIDI data in a nice/intuitive way.

Macro stlgtm:
matching names.. makes my little heart jump

In D12998#275744, @tcberner wrote:

matching names.. makes my little heart jump

Tobias, mat asked for directory to be named with underscore. What should be changed?

That's exactly what I wanted too :)

As long as @mat has nothing more to add this is good to go from my side.

This revision is now accepted and ready to land.Nov 24 2017, 8:29 PM

Closed by commit rP454848: New port: multimedia/py-pretty_midi: Utility functions for handling MIDI data… (authored by yuri). · Explain WhyNov 24 2017, 8:33 PM

This revision was automatically updated to reflect the committed changes.

You have to read my accepting comments, not just commit :)

In D12998#275767, @tcberner wrote:

You have to read my accepting comments, not just commit :)

Sorry!

Revision Contents
Changeset List

Path

Size

head/

multimedia/

1 line

py-pretty_midi/

22 lines

3 lines

4 lines

Diff 35733

head/multimedia/Makefile

Loading...

head/multimedia/py-pretty_midi/Makefile

Loading...

head/multimedia/py-pretty_midi/distinfo

Loading...

head/multimedia/py-pretty_midi/pkg-descr

Loading...