Merge lp:~jbicha/ubuntu-terminal-app/fix-binary-name-conflict into lp:ubuntu-terminal-app

Proposed by Jeremy Bícha
Status: Needs review
Proposed branch: lp:~jbicha/ubuntu-terminal-app/fix-binary-name-conflict
Merge into: lp:ubuntu-terminal-app
Diff against target: 105 lines (+14/-14)
4 files modified
CMakeLists.txt (+2/-2)
manifest.json.in (+1/-1)
src/app/CMakeLists.txt (+5/-5)
ubuntu-terminal-app.1 (+6/-6)
To merge this branch: bzr merge lp:~jbicha/ubuntu-terminal-app/fix-binary-name-conflict
Reviewer Review Type Date Requested Status
Ubuntu Terminal Developers Pending
Review via email: mp+321786@code.launchpad.net

Description of the change

/usr/bin/terminal is too generic of a name and it turns out something else in Ubuntu already ships that file.

To post a comment you must log in.

Unmerged revisions

359. By Jeremy Bícha

rename binary from too-generic 'terminal' (LP: #1651161)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2017-01-25 15:20:00 +0000
+++ CMakeLists.txt 2017-04-03 20:56:36 +0000
@@ -15,7 +15,7 @@
15option(INSTALL_TESTS "Install the tests on make install" on)15option(INSTALL_TESTS "Install the tests on make install" on)
16option(CLICK_MODE "Installs to a contained location" on)16option(CLICK_MODE "Installs to a contained location" on)
1717
18set(APP_NAME terminal)18set(APP_NAME ubuntu-terminal-app)
19set(DESKTOP_FILE "${PROJECT_NAME}.desktop")19set(DESKTOP_FILE "${PROJECT_NAME}.desktop")
20set(APP_HARDCODE ubuntu-terminal-app)20set(APP_HARDCODE ubuntu-terminal-app)
21set(MAIN_QML ubuntu-terminal-app.qml)21set(MAIN_QML ubuntu-terminal-app.qml)
@@ -71,7 +71,7 @@
71 set(DATA_DIR ${CMAKE_INSTALL_DATADIR}/${APP_NAME})71 set(DATA_DIR ${CMAKE_INSTALL_DATADIR}/${APP_NAME})
72 set(ICON ${CMAKE_INSTALL_PREFIX}/${DATA_DIR}/${ICON_FILE})72 set(ICON ${CMAKE_INSTALL_PREFIX}/${DATA_DIR}/${ICON_FILE})
73 set(DESKTOP_DIR ${CMAKE_INSTALL_DATADIR}/applications)73 set(DESKTOP_DIR ${CMAKE_INSTALL_DATADIR}/applications)
74 install(FILES ${CMAKE_CURRENT_SOURCE_DIR}/terminal.1 DESTINATION ${CMAKE_INSTALL_DATADIR}/man/man1)74 install(FILES ${CMAKE_CURRENT_SOURCE_DIR}/ubuntu-terminal-app.1 DESTINATION ${CMAKE_INSTALL_DATADIR}/man/man1)
75endif(CLICK_MODE)75endif(CLICK_MODE)
7676
77file(GLOB_RECURSE I18N_SRC_FILES77file(GLOB_RECURSE I18N_SRC_FILES
7878
=== modified file 'manifest.json.in'
--- manifest.json.in 2017-01-25 15:20:00 +0000
+++ manifest.json.in 2017-04-03 20:56:36 +0000
@@ -5,7 +5,7 @@
5 "hooks": {5 "hooks": {
6 "terminal": {6 "terminal": {
7 "apparmor": "terminal.apparmor",7 "apparmor": "terminal.apparmor",
8 "desktop": "ubuntu-terminal-app.desktop"8 "desktop": "com.ubuntu.terminal.desktop"
9 }9 }
10 },10 },
11 "icon": "terminal64.png",11 "icon": "terminal64.png",
1212
=== modified file 'src/app/CMakeLists.txt'
--- src/app/CMakeLists.txt 2016-12-09 13:21:01 +0000
+++ src/app/CMakeLists.txt 2017-04-03 20:56:36 +0000
@@ -9,10 +9,10 @@
9 ${QML_SRCS}9 ${QML_SRCS}
10)10)
1111
12add_executable(terminal ${terminal_SRCS})12add_executable(ubuntu-terminal-app ${terminal_SRCS})
1313
14qt5_use_modules(terminal Gui Qml Quick Widgets)14qt5_use_modules(ubuntu-terminal-app Gui Qml Quick Widgets)
15target_link_libraries(terminal stdc++)15target_link_libraries(ubuntu-terminal-app stdc++)
1616
17if(NOT "${CMAKE_CURRENT_SOURCE_DIR}" STREQUAL "${CMAKE_CURRENT_BINARY_DIR}")17if(NOT "${CMAKE_CURRENT_SOURCE_DIR}" STREQUAL "${CMAKE_CURRENT_BINARY_DIR}")
18add_custom_target(terminal-qmlfiles ALL18add_custom_target(terminal-qmlfiles ALL
@@ -23,7 +23,7 @@
2323
24install(DIRECTORY qml DESTINATION ${DATA_DIR})24install(DIRECTORY qml DESTINATION ${DATA_DIR})
25if(CLICK_MODE)25if(CLICK_MODE)
26 install(TARGETS terminal DESTINATION ${BIN_DIR})26 install(TARGETS ubuntu-terminal-app DESTINATION ${BIN_DIR})
27else()27else()
28 install(TARGETS terminal RUNTIME DESTINATION bin)28 install(TARGETS ubuntu-terminal-app RUNTIME DESTINATION bin)
29endif()29endif()
3030
=== renamed file 'terminal.1' => 'ubuntu-terminal-app.1'
--- terminal.1 2016-09-16 03:01:32 +0000
+++ ubuntu-terminal-app.1 2017-04-03 20:56:36 +0000
@@ -2,7 +2,7 @@
2.\" First parameter, NAME, should be all caps2.\" First parameter, NAME, should be all caps
3.\" Second parameter, SECTION, should be 1-8, maybe w/ subsection3.\" Second parameter, SECTION, should be 1-8, maybe w/ subsection
4.\" other parameters are allowed: see man(7), man(1)4.\" other parameters are allowed: see man(7), man(1)
5.TH TERMINAL 1 "September 16, 2016"5.TH UBUNTU-TERMINAL-APP 1 "September 16, 2016"
6.\" Please adjust this date whenever revising the manpage.6.\" Please adjust this date whenever revising the manpage.
7.\"7.\"
8.\" Some roff macros, for reference:8.\" Some roff macros, for reference:
@@ -16,16 +16,16 @@
16.\" .sp <n> insert n+1 empty lines16.\" .sp <n> insert n+1 empty lines
17.\" for manpage-specific macros, see man(7)17.\" for manpage-specific macros, see man(7)
18.SH NAME18.SH NAME
19terminal \- is a terminal emulation application.19ubuntu-terminal-app \- is a terminal emulation application.
20.SH SYNOPSIS20.SH SYNOPSIS
21.sp21.sp
22.nf22.nf
23\fIterminal\fR [\-p|\-\-phone] [\-\-workdir <dir>]23\fIubuntu-terminal-app\fR [\-p|\-\-phone] [\-\-workdir <dir>]
24 [\-\-forceAuth <true/false>] [\-t|\-\-tablet] [\-h|\-\-help] [\-I <path>]24 [\-\-forceAuth <true/false>] [\-t|\-\-tablet] [\-h|\-\-help] [\-I <path>]
25.nf25.nf
26.sp26.sp
27.SH DESCRIPTION27.SH DESCRIPTION
28.B terminal28.B ubuntu-terminal-app
29is a terminal emulation application that you can use to access a unix shell.29is a terminal emulation application that you can use to access a unix shell.
30.SH OPTIONS30.SH OPTIONS
31.TP31.TP
@@ -50,7 +50,7 @@
50.B \-\-workdir <dir>50.B \-\-workdir <dir>
51Change working directory to 'dir'51Change working directory to 'dir'
52.SH AUTHOR52.SH AUTHOR
53Terminal was written by Filippo Scognamiglio <flscogna@gmail.com>. The webpage for this package can be found at53ubuntu-terminal-app was written by Filippo Scognamiglio <flscogna@gmail.com>. The webpage for this package can be found at
54http://launchpad.net/ubuntu-terminal-app54https://launchpad.net/ubuntu-terminal-app
55.PP55.PP
56This manual page was written by Luke Yelavich <themuso@ubuntu.com>, for the Ubuntu project (but may be used by others).56This manual page was written by Luke Yelavich <themuso@ubuntu.com>, for the Ubuntu project (but may be used by others).

Subscribers

People subscribed via source and target branches