Page MenuHomeFreeBSD

audio/mixxx update to 2.1.4
AbandonedPublic

Authored by acm on Apr 24 2018, 4:07 AM.
Referenced Files
Unknown Object (File)
Fri, Apr 12, 2:03 PM
Unknown Object (File)
Mon, Apr 8, 10:16 AM
Unknown Object (File)
Dec 20 2023, 2:52 AM
Unknown Object (File)
Nov 23 2023, 1:09 PM
Unknown Object (File)
Nov 23 2023, 11:47 AM
Unknown Object (File)
Nov 23 2023, 9:45 AM
Unknown Object (File)
Nov 15 2023, 8:06 AM
Unknown Object (File)
Nov 13 2023, 8:03 AM
Subscribers

Details

Reviewers
None
Group Reviewers
portmgr
Summary
  • Add qt4|qt5 FLAVORS
Test Plan

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

mixxx/Makefile
1 ↗(On Diff #41790)

The only case where you can remove this line is if the person in it asked you to. In all other cases, it is write-only.

7 ↗(On Diff #41790)

The distribution file name is too generic, please use DIST_SUBDIR.

33 ↗(On Diff #41790)

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

35–36 ↗(On Diff #41790)

Wrong place in the Makefile.

38 ↗(On Diff #41790)

Wrong place in the Makefile.

41–53 ↗(On Diff #41790)

Wrong place in the Makefile.

63–66 ↗(On Diff #41790)

Wrong place in the Makefile.

acm marked 7 inline comments as done.
  • Apply changes according mat commentaries
Makefile
6–7

Sorry, missed the fact that you were not using USE_GITHUB like you should, here, add:

DISTVERSIONPREFIX=release-

and later, add:

USE_GITHUB=yes
GH_ACCOUNT=mixxxdj
70

Remove.

96–102

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

acm marked 3 inline comments as done.
  • Apply changes according mat commentaries
  • Update patch file with right version
mixxx/Makefile
5 ↗(On Diff #43347)

You should probably be using DISTVERSION.

8 ↗(On Diff #43347)

Not required, could be removed.

35–36 ↗(On Diff #43347)

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

83–96 ↗(On Diff #43347)

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

acm retitled this revision from audio/mixxx update to 2.1.0 to audio/mixxx update to 2.1.1.Aug 3 2018, 3:07 PM
  • Apply changes accord to mat commentaries
  • Migrate to new QT helpers
Makefile
33–34

You should probably help to set FLAVOR here, you could drop all :U* dances later, see https://www.freebsd.org/doc/en/books/porters-handbook/flavors-using.html

46

Default, please remove.

  • Apply mat's suggestions
acm marked 2 inline comments as done.Aug 6 2018, 6:44 PM
Makefile
36–37

Not exactly sure why using flavors helpers for this, it is the same value for both.

  • Add missing items to flavor_CONFLICTS
acm marked an inline comment as done.Aug 7 2018, 7:24 AM
acm marked 2 inline comments as done.
acm marked 2 inline comments as done.
mixxx/Makefile
46 ↗(On Diff #46373)

There is a release-2.1.1 you can use:

DISTVERSIONPREFIX=  release-
acm retitled this revision from audio/mixxx update to 2.1.1 to audio/mixxx update to 2.1.3.
  • Update to 2.1.3
  • Add DISTVERSIONPREFIX
  • Add CONFLICTS
acm marked an inline comment as done.Aug 30 2018, 5:44 AM
mixxx/Makefile
5 ↗(On Diff #47473)

This should probably be DISTVERSION.

18 ↗(On Diff #47473)

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

acm retitled this revision from audio/mixxx update to 2.1.3 to audio/mixxx update to 2.1.4.
  • Update to 2.1.4
  • Some minor changes
acm marked 2 inline comments as done.Oct 24 2018, 6:04 AM
  • Remove FLAVOR support. QT4 will be deprecated soon. it'll use QT instead of