[Bug 658754] Review Request: CUBRID - a very fast and reliable open source SQL database server.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=658754

--- Comment #12 from Esen Sagynov <esen.sagynov@xxxxxxx> ---
Hello Cristian,

First of all, thank you for your continuos support on IRC. You feedback was
very valuable for us to improve our CUBRID spec.

Below please find our last changes we've made according to your comments.

1) Regarding macros, we've replaced %{_prefix}/share/ with %{_datarootdir} and
%{_prefix}/include -> %{_includedir}

2) "Some files under /usr/include and share are owned by cubrid even if they
shouldn't." /usr/include directory is not owned by cubrid user. Only share dir
was. But when we tried to chown root:root, we found out that cubrid master
service checks if the files are owned by the user who is running the process.
When root:root, the cubrid master service fails to load saying “Current user
does not match CUBRID user”. Therefore, we had to roll back and chown
cubrid:cubrid the shared files. We have reported this issues in our Issue
Tracker, so we will look into it in the future version.

4) Regarding "devel-file-in-non-devel-package", like I explained before some
libs must be in the main package because CUBRID uses them, otherwise, CUBRID
will not work; we moved all libs to devel package which were possible to move.

5) Regarding, "put all commands under one giant command command like git does",
the "cubrid" command actually does exactly that. This is that giant command.

6) “strip --strip-unneded”: we have completely removed strip commands, which
now gives us only warnings saying "W: unstripped-binary-or-object".

7) This allows us at least have the debuginfo package created and it has no
errors related to "empty-debuginfo-package".

8) We now keep changelogs up to date. Also spec release number is always UP'ed
and file version is updated if necessary.

9) According to your suggestion, we added conditionals to handle different Java
versions for different versions of Fedora.

10) For new Fedora we use systemd now, and for old including RH distributions
use "SysV scripts".

11) "Replace %{libdir} with the standard %{_libdir} macro". As I explained
before, we cannot do this in this version because we have a hardcoded /usr/lib
path which is used even on 64-bit system architecture. We will fix this in the
CUBRID core in the future version.

12) We no longer start the program/service after installation.

13) "If possible, use a private /tmp in your systemd service unit". We didn't
add "PrivateTmp=true" because "cubrid master" process failed to start if this
line was set in cubrid.service.

I am glad that so far CUBRID package is being successfully built at Fedora.
Please let me know if there is anything we can do more to improve the CUBRID
spec.

Thank you Cristian!

Regards,
Esen Sagynov.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]