dbf2mysql and hardcoded mysql-client version dependency
Miroslav Lachman
000.fbsd at quip.cz
Sun Nov 28 23:14:24 UTC 2010
Matthew Seaman wrote:
> On 28/11/2010 19:01, Miroslav Lachman wrote:
>> Can somebody explain me, why databases/dbf2mysql has hardcoded
>> dependency to mysql323-client?
>>
>> The Makefile contains:
>>
>> LIB_DEPENDS= mysqlclient.10:${PORTSDIR}/databases/mysql323-client
>>
>> But it is unbuildable
>
> The sources to dbf2mysql were last modified around 2000, when mysql-3.2x
> was current. Since then, it seems to have been pretty much abandoned.
>
> A good candidate for deletion from the ports on grounds of obsolescence.
I attached two patches. Makefile.patch is for port's Makefile and
patch-mysql2dbf should be placed into files port directory.
Then it works with MySQL 5.0.x (tested with mysql-client-5.0.90)
I tested just a dbf2mysql command, patch for mysql2dbf is needed to
compile, but I did not tested it as I do not have any apps to test
produced dbf file.
Should I send a PR for this fix?
Miroslav Lachman
-------------- next part --------------
--- Makefile.orig 2010-11-28 18:34:26.000000000 +0100
+++ Makefile 2010-11-28 19:11:05.000000000 +0100
@@ -18,7 +18,7 @@
MAINTAINER= ports at FreeBSD.org
COMMENT= Programs to convert .dbf files to MySQL tables and vice versa
-LIB_DEPENDS= mysqlclient.10:${PORTSDIR}/databases/mysql323-client
+USE_MYSQL= yes
PORTDOCS= README
-------------- next part --------------
--- mysql2dbf.c.orig 2010-11-28 19:46:34.000000000 +0100
+++ mysql2dbf.c 2010-11-28 21:23:22.000000000 +0100
@@ -1,6 +1,11 @@
/* utility to read out an mySQL-table, and store it into a DBF-file
M. Boekhold (boekhold at cindy.et.tudelft.nl) April 1996
+
+ M. Lachman Nov 2010
+ Replaced mysql_connect with mysql_real_connect, SQLsock isn't used anymore
+ Can be compiled with MySQL client 5.0.x
+
*/
#include <stdio.h>
@@ -47,7 +52,7 @@
int main(int argc, char **argv) {
int i;
- MYSQL *SQLsock,mysql;
+ MYSQL mysql;
extern int optind;
extern char *optarg;
char *query = NULL;
@@ -148,7 +153,9 @@
printf("Making connection with mySQL-server\n");
}
- if (!(SQLsock = mysql_connect(&mysql,host,user,pass))) {
+ mysql_init(&mysql);
+
+ if (!mysql_real_connect(&mysql,host,user,pass, dbase, 0, NULL,0)) {
fprintf(stderr, "Couldn't get a connection with the ");
fprintf(stderr, "designated host!\n");
fprintf(stderr, "Detailed report: %s\n", mysql_error(&mysql));
@@ -157,37 +164,24 @@
exit(1);
}
- if (verbose > 1) {
- printf("Selecting database\n");
- }
-
- if ((mysql_select_db(SQLsock, dbase)) == -1) {
- fprintf(stderr, "Couldn't select database %s.\n", dbase);
- fprintf(stderr, "Detailed report: %s\n", mysql_error(SQLsock));
- close(dbh->db_fd);
- free(dbh);
- mysql_close(SQLsock);
- exit(1);
- }
-
if (verbose > 1) {
printf("Sending query\n");
}
- if (mysql_query(SQLsock, query) == -1) {
+ if (mysql_query(&mysql, query) == -1) {
fprintf(stderr, "Error sending query.\nDetailed report: %s\n",
- mysql_error(SQLsock));
+ mysql_error(&mysql));
if (verbose > 1) {
fprintf(stderr, "%s\n", query);
}
- mysql_close(SQLsock);
+ mysql_close(&mysql);
close(dbh->db_fd);
free(dbh);
exit(1);
}
- qres = mysql_store_result(SQLsock);
+ qres = mysql_store_result(&mysql);
numfields = mysql_num_fields(qres);
numrows = mysql_num_rows(qres);
@@ -303,7 +297,7 @@
}
mysql_free_result(qres);
- mysql_close(SQLsock);
+ mysql_close(&mysql);
close(dbh->db_fd);
free(dbh);
exit(0);
More information about the freebsd-ports
mailing list