Skip to content

Decouple tbench and dbench #508

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

Merged
merged 1 commit into from
Apr 23, 2025
Merged

Decouple tbench and dbench #508

merged 1 commit into from
Apr 23, 2025

Conversation

Suneeth-D
Copy link

The updated source URL for the dbench package that was introduced in 31dc80f was causing a build error while attempting to install the tbench. In order to fix that, efforts to decouple tbench and dbench are taken.

The following changes are introduced in this commit:-

1) Decouple tbench and dbench such that tbench's pkg/ dir and
parse file unlink from dbench's

2) Change the source url from 'https://github.com/sahlberg/dbench'
to 'https://www.samba.org/ftp/tridge/dbench'

3) Add libtirpc-dev to tbench/depends-dev

4) Add unit tests under spec/stats

The updated source URL for the dbench package that was introduced in
31dc80f was causing a build error while attempting to install
the tbench. In order to fix that, efforts to decouple tbench and
dbench are taken.

The following changes are introduced in this commit:-

	1) Decouple tbench and dbench such that tbench's pkg/ dir and
	parse file unlink from dbench's

	2) Change the source url from 'https://github.com/sahlberg/dbench'
	to 'https://www.samba.org/ftp/tridge/dbench'

	3) Add libtirpc-dev to tbench/depends-dev

	4) Add unit tests under spec/stats

Signed-off-by: Suneeth D <Suneeth.D@amd.com>
Reviewed-by: Srikanth Aithal <saithal@amd.com>
@rli9 rli9 merged commit 6f5bdc4 into intel:master Apr 23, 2025
12 checks passed
@rli9
Copy link

rli9 commented Apr 23, 2025

thanks for the patch

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