Skip to content
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.

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

Fix my_print_default client #1566

Closed
wants to merge 2 commits into from

Conversation

Sp1l
Copy link
Contributor

@Sp1l Sp1l commented May 31, 2020

my_print_default is a client app, it is required by scripts/mytop

@an3l
Copy link
Collaborator

an3l commented Jun 1, 2020

Probably mytop will not be part of client package anymore, PR #1565 (MDEV-22552), so no need for my_print_defaults change.

@an3l an3l self-assigned this Jun 2, 2020
@an3l an3l added this to the 10.5 milestone Jun 2, 2020
@ottok
Copy link
Contributor

ottok commented Jun 2, 2020

Checked out the situation in deb packages: my_print_defaults is part of the mariadb-server-10.5 package and used among others by the logrotate script (until #1556 removes it).

The mariadb-server-10.5 package depends on mariadb-client-10.5 so the change should be OK.

ottok added a commit to ottok/mariadb that referenced this pull request Jun 5, 2020
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_deafult is always available when mytop
is installed.

The my_print_defaults is also required by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Related: MariaDB#1566
ottok added a commit to ottok/mariadb that referenced this pull request Jun 5, 2020
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_deafult is always available when mytop
is installed.

The my_print_defaults is also required by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Related: MariaDB#1566
an3l pushed a commit that referenced this pull request Jun 6, 2020
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_deafult is always available when mytop
is installed.

The my_print_defaults is also required by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Related: #1566
an3l pushed a commit that referenced this pull request Jun 6, 2020
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_deafult is always available when mytop
is installed.

The my_print_defaults is also required by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Related: #1566

Closes 1581
an3l pushed a commit that referenced this pull request Aug 2, 2020
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_deafult is always available when mytop
is installed.

The my_print_defaults is also required by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Related: #1566

Closes 1581
ottok added a commit to ottok/mariadb that referenced this pull request Aug 7, 2020
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_deafult is always available when mytop
is installed.

The my_print_defaults is also required by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Related: MariaDB#1566
an3l pushed a commit that referenced this pull request Mar 17, 2021
an3l pushed a commit that referenced this pull request Mar 17, 2021
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_deafult is always available when mytop
is installed.

The my_print_defaults is also required by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Related: #1566

Closes PR #1581
grooverdan pushed a commit that referenced this pull request Mar 18, 2021
grooverdan pushed a commit that referenced this pull request Mar 18, 2021
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_deafult is always available when mytop
is installed.

The my_print_defaults is also required by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Related: #1566

Closes PR #1581
grooverdan pushed a commit that referenced this pull request Mar 18, 2021
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_deafult is always available when mytop
is installed.

The my_print_defaults is also required by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Related: #1566

Closes PR #1581
grooverdan pushed a commit that referenced this pull request Mar 19, 2021
grooverdan pushed a commit that referenced this pull request Mar 19, 2021
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_deafult is always available when mytop
is installed.

The my_print_defaults is also required by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Related: #1566

Closes PR #1581
grooverdan pushed a commit that referenced this pull request Mar 28, 2021
grooverdan pushed a commit that referenced this pull request Mar 28, 2021
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_deafult is always available when mytop
is installed.

The my_print_defaults is also required by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Related: #1566

Closes PR #1581
@an3l
Copy link
Collaborator

an3l commented Mar 28, 2021

Thanks pushed with 4f8945e on 10.6

@an3l an3l closed this Mar 28, 2021
ottok added a commit to ottok/mariadb that referenced this pull request Apr 2, 2021
There are many programs that actually belong to the client or
even client core package, as they should be fully usable without
a locally installed server.

Examples:
- The my_print_default is required by mytop which is also in the MariaDB
  client package, which in turn requires the client core package.
  This way it is ensured my_print_deafult is always available when mytop
  is installed.

  The my_print_defaults is also required by the server logrotate files
  and mysqld_safe script etc, but this change is fine, since the server
  core package always depends on the MariaDB client core package anyway
  and they are both installed on server installations.

  Closes: MariaDB#1566, MariaDB#1581 (both PRs to move my_print_defaults)
ottok added a commit to ottok/mariadb that referenced this pull request Apr 4, 2021
There are many programs that actually belong to the client or
even client core package, as they should be fully usable without
a locally installed server.

Examples:
- The my_print_default is required by mytop which is also in the MariaDB
  client package, which in turn requires the client core package.
  This way it is ensured my_print_deafult is always available when mytop
  is installed.

  The my_print_defaults is also required by the server logrotate files
  and mysqld_safe script etc, but this change is fine, since the server
  core package always depends on the MariaDB client core package anyway
  and they are both installed on server installations.

  Closes: MariaDB#1566, MariaDB#1581 (both PRs to move my_print_defaults)
ottok added a commit to ottok/mariadb that referenced this pull request Apr 4, 2021
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_default is always available when mytop
is installed.

The my_print_defaults is also requires by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Closes: MariaDB#1566, MariaDB#1581
ottok added a commit to ottok/mariadb that referenced this pull request Apr 11, 2021
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_default is always available when mytop
is installed.

The my_print_defaults is also requires by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Closes: MariaDB#1566, MariaDB#1581
ottok added a commit to ottok/mariadb that referenced this pull request Apr 15, 2021
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_default is always available when mytop
is installed.

The my_print_defaults is also requires by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Closes: MariaDB#1566, MariaDB#1581
grooverdan pushed a commit that referenced this pull request Apr 16, 2021
The my_print_default is required by mytop which is also in the MariaDB
client package, which in turn requires the client core package.
This way it is ensured my_print_default is always available when mytop
is installed.

The my_print_defaults is also requires by the server logrotate files
and mysqld_safe script etc, but this change is fine, since the server
core package always depends on the MariaDB client core package anyway
and they are both installed on server installations.

Closes: #1566, #1581
an3l pushed a commit to an3l/server that referenced this pull request Jun 23, 2021
Make `my_print_defaults` a client app (required by `mytop`).

Closes MariaDB#1566

Reviewed by: serg@mariadb.com, anel@mariadb.org
an3l pushed a commit to an3l/server that referenced this pull request Oct 25, 2021
Make `my_print_defaults` a client app (required by `mytop`).

Closes MariaDB#1566

Reviewed by: serg@mariadb.com, anel@mariadb.org
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants