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

listmgr: Remove final reference of my_bool #137

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

arshad512
Copy link
Contributor

Function db_connect() which has a reference to
data type 'my_pool' was added via commit 2d1de16
on 03 Aug 2010.

Subsequently, via patch 6c174bd2a59d dated 24 Feb 2017
all "my_bool" defined was replaced via data type bool.

However, the current build is failing with 'unknown type' as
below under machine configuration show below that.

*** Build error ***
mysql_wrapper.c: In function ‘db_connect’:
error: unknown type name ‘my_bool’; did you mean ‘bool’?
my_bool reconnect = 1;


$ gcc --version | grep GCC
gcc (GCC) 8.5.0 20210514 (Red Hat 8.5.0-18)

$ uname -r
4.18.0-425.3.1.el8_lustre.x86_64

$ cat /etc/rocky-release
Rocky Linux release 8.7 (Green Obsidian)

$ git log --format=oneline -1
509584b (origin/master, origin/HEAD, master)
policies: implement asc/desc modifier for sort order

This patch replaces single reference to "my_bool" with
bool making the build pass and looks like type 'my_bool'
is not used anymore.

Change-Id: I88d8dcdbb49370e8d09d669c523d69a277adc148

Function db_connect() which has a reference to
data type 'my_pool' was added via commit 2d1de16
on 03 Aug 2010.

Subsequently, via patch 6c174bd2a59d dated 24 Feb 2017
all "my_bool" defined was replaced via data type bool.

However, the current build is failing with 'unknown type'
as below under machine configuration show below that.

*** Build error ***
mysql_wrapper.c: In function ‘db_connect’:
error: unknown type name ‘my_bool’; did you mean ‘bool’?
     my_bool reconnect = 1;
*******************

$ gcc --version | grep GCC
gcc (GCC) 8.5.0 20210514 (Red Hat 8.5.0-18)

$ uname -r
4.18.0-425.3.1.el8_lustre.x86_64

$ cat /etc/rocky-release
Rocky Linux release 8.7 (Green Obsidian)

$ git log --format=oneline -1
509584b (origin/master, origin/HEAD, master) \
	policies: implement asc/desc modifier for sort order

This patch replaces _single_ reference to "my_bool"
with bool making the build pass and looks like type
'my_bool' is not used anymore.

Change-Id: I88d8dcdbb49370e8d09d669c523d69a277adc148
Signed-off-by: Arshad Hussain <[email protected]>
@arshad512
Copy link
Contributor Author

Ping for review. Thanks

@tl-cea
Copy link
Member

tl-cea commented Aug 30, 2024

Thank you for reporting and contributing.

When building on el8.8 I don't get such error.
Was it with mariadb or another DB package?

@arshad512
Copy link
Contributor Author

Sorry, I should have mentioned in the bug info. It is influxdb.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants