[MDEV-12096] MariaDB database client library (libmariadb3) .deb package symlinks errors Created: 2017-02-20 Updated: 2017-03-11 Resolved: 2017-03-11 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Platform Debian |
| Affects Version/s: | 10.2.4 |
| Fix Version/s: | 10.2.5, 10.3.0 |
| Type: | Bug | Priority: | Major |
| Reporter: | Peter Schmidiger | Assignee: | Otto Kekäläinen |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | buildbot, packaging | ||
| Environment: |
Ubuntu 14.04.5 LTS x86_64 |
||
| Description |
|
The versioned client library file symlinks targets are absolute paths to a non-existing file: Package:
|
| Comments |
| Comment by Vladislav Vaintroub [ 2017-02-24 ] |
|
possibly a consequence of https://github.com/MariaDB/server/commit/9b10b4181f70634c9fab96ff0a225147b64396d7 |
| Comment by Otto Kekäläinen [ 2017-03-08 ] |
|
The commit message "update libmariadb, and fix debian packaging for client libs" is void of all useful information. It does not state why you did something and what the problem was it fixes or what it attempts to do. There is no comments embedded into the code itself either. So if is difficult to review if your changes were correct or not when there is no messages of intention. The git subject just states the obivious that is already visible in the diff. The bug report does not either have any context, link to an actually failing build etc so it is hard to debug with this as a starting point. Please provide more information. |
| Comment by Vladislav Vaintroub [ 2017-03-08 ] |
|
I do not remember, sorry. you can revert and see where it fail. that's the easiest way to do. As for "why", we update libmariadb submodule, and this brought a couple of errors, that I had to fix without any idea of the script that I touched. debian packaging is not my strength. The commit that probably broke it is this : https://github.com/MariaDB/mariadb-connector-c/commit/e34a595ce2bff180518df42023dad0b6188b6233 . Since we package with cpack elsewhere, it is only carefully written debian scripts that can ever break. |
| Comment by Otto Kekäläinen [ 2017-03-09 ] |
|
Hello! Debian packaging is sometimes a bit tricky and I completely understand if you don't know exactly what you are doing, no blame there. However writing git commits should be your everyday bread and there is no excuse to commit stuff without meaningful commit messages. In the slides 17-20 at https://www.slideshare.net/ottokekalainen/git-best-practices-2016 you can read what I recommend about git commit messages in general. I will give a shot at building current 10.2 master, see what fails and perhaps fix this. |
| Comment by Otto Kekäläinen [ 2017-03-11 ] |
|
Fixed for 10.3 in https://github.com/MariaDB/server/commit/f7740b6604ae324be20feab192008f015c139498 I waited for buildbot run to finish over the night, but it didn't and I don't have time to prolong this so I pushed to masters now. The change is pretty safe and very unlikely to produce regressions. |