Skip to content

Document TARGET naming mechanisms in Jamulus.pro #3850

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mcfnord wants to merge 1 commit into
base: main
Choose a base branch
from
+10 −0
Open

Document TARGET naming mechanisms in Jamulus.pro #3850

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions Jamulus.pro
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,16 @@ lessThan(QT_MAJOR_VERSION, 5) | equals(QT_MAJOR_VERSION, 5) : lessThan(QT_MINOR_
error(Jamulus requires at least Qt5.12. See https://github.com/jamulussoftware/jamulus/pull/3288)
}

# Target naming. The two Debian packages get their binary names by different means
# (see linux/debian/rules): the desktop build passes CONFIG+=noupcasename, handled
# here, while the headless build passes TARGET=jamulus-headless on the qmake command
# line. TARGET selects the installed usr/bin/<TARGET> listed in debian/*.install and
# is substituted into Exec= in the generated .desktop files, and it has to keep
# matching the hardcoded ExecStart=/usr/bin/jamulus-headless in the headless unit.
# Note the assignment below overrides any TARGET= given on the command line, so
# adding noupcasename to the headless build would silently rename its binary to
# "jamulus" and break that package.

# use target name which does not use a capital letter at the beginning
contains(CONFIG, "noupcasename") {
message(The target name is jamulus instead of Jamulus.)
Expand Down
Loading