Re: [pgagent] Instatallation destination?

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: [pgagent] Instatallation destination?
Дата
Msg-id CA+OCxowjrfzO3sCAEgaCcttWvww=bj8vxpogSb4L0w-22dVWiA@mail.gmail.com
обсуждение исходный текст
Ответ на [pgagent] Instatallation destination?  (Christoph Berg <myon@debian.org>)
Ответы Re: [pgagent] Instatallation destination?  (Neel Patel <neel.patel@enterprisedb.com>)
Список pgadmin-support
Hi

On Fri, Jul 20, 2018 at 10:00 AM, Christoph Berg <myon@debian.org> wrote:
Hi,

in the Debian package I notice that CMakeLists.txt installs README and
LICENSE to /usr (no subdirectory!), and pgagent.sql to /usr/share.
The first is clearly wrong, and the second somewhat debatable. As
pgagent.sql is not really required (extension support), I added this
patch:

Urgh, yeah.
 

Don't install README and LICENSE which would otherwise end up in /usr,
and don't install pgagent.sql to /usr/share.

--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -144,11 +144,11 @@ IF (WIN32)
     INSTALL(FILES ${pgagent_SOURCE_DIR}/sql/pgagent.sql DESTINATION .)
 ELSE(WIN32)
     INSTALL(TARGETS pgagent DESTINATION bin)
-    INSTALL(FILES ${pgagent_SOURCE_DIR}/sql/pgagent.sql DESTINATION share)
+    #INSTALL(FILES ${pgagent_SOURCE_DIR}/sql/pgagent.sql DESTINATION share)
 ENDIF(WIN32)

-INSTALL(FILES ${pgagent_SOURCE_DIR}/README DESTINATION .)
-INSTALL(FILES ${pgagent_SOURCE_DIR}/LICENSE DESTINATION .)
+#INSTALL(FILES ${pgagent_SOURCE_DIR}/README DESTINATION .)
+#INSTALL(FILES ${pgagent_SOURCE_DIR}/LICENSE DESTINATION .)

 # Extension
 IF(PG_EXTENSION)


What would be the proper fix?

I suspect we need to do something similar to PostgreSQL which (iirc) changes the layout slightly if the prefix includes "pgsql" or "postgresql". The current behaviour looks reasonable in that case; e.g.

Install the project...

-- Install configuration: "Release"

-- Installing: /usr/local/pgagent/bin/pgagent

-- Installing: /usr/local/pgagent/share/pgagent.sql

-- Installing: /usr/local/pgagent/./README

-- Installing: /usr/local/pgagent/./LICENSE

-- Installing: /Library/PostgreSQL/11/share/postgresql/extension/pgagent--4.0.sql

-- Installing: /Library/PostgreSQL/11/share/postgresql/extension/pgagent.control

-- Up-to-date: /Library/PostgreSQL/11/share/postgresql/extension/pgagent.control

-- Installing: /Library/PostgreSQL/11/share/postgresql/extension/pgagent--3.4--4.0.sql

-- Installing: /Library/PostgreSQL/11/share/postgresql/extension/pgagent--unpackaged--4.0.sql

However, if the prefix does not "pgagent" or "pgAgent", perhaps it should look more like:

Install the project...

-- Install configuration: "Release"

-- Installing: /usr/local/bin/pgagent

-- Installing: /usr/local/share/pgagent.sql

-- Installing: /usr/local/share/pgagent/README

-- Installing: /usr/local/share/pgagent/LICENSE

-- Installing: /Library/PostgreSQL/11/share/postgresql/extension/pgagent--4.0.sql

-- Installing: /Library/PostgreSQL/11/share/postgresql/extension/pgagent.control

-- Up-to-date: /Library/PostgreSQL/11/share/postgresql/extension/pgagent.control

-- Installing: /Library/PostgreSQL/11/share/postgresql/extension/pgagent--3.4--4.0.sql

-- Installing: /Library/PostgreSQL/11/share/postgresql/extension/pgagent--unpackaged--4.0.sql


 Does that look about right?

Assuming so, can you implement please Neel?

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

В списке pgadmin-support по дате отправления:

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: [pgagent] patch: link with -pthread
Следующее
От: Christoph Berg
Дата:
Сообщение: [pgagent] regression test