gmane.comp.db.mdb-tools.devel

namespace: reserved C++ keyword

Nirgal — 2012-02-11T17:45:23

I renamed namespace into dbnamespace.
See https://github.com/brianb/mdbtools/commit/7634e329c740825c97652175bcad16a1a58688ae

------------------------------------------------------------------------------
Virtualization & Cloud Management Using Capacity Planning
Cloud computing makes use of virtualization - but cloud computing 
also focuses on allowing computing to be delivered as a service.
http://www.accelacomm.com/jaw/sfnl/114/51521223/

-Wall fixing

Nirgal — 2012-02-02T19:29:56

Sorry for the spam, one more:

https://github.com/brianb/mdbtools/commit/b7200c9b576cf8c22beae63927c6fedc2e6816c0
contains some trivial fixes for -Wall compilation

------------------------------------------------------------------------------
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d

Re: 2 more patches

Nirgal — 2012-02-02T18:54:47

Commited.

------------------------------------------------------------------------------
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d

man: removed <> from synopsis lines

Nirgal — 2012-02-02T18:19:39

I think <> should not be present in sysnopsis lines in man pages since txt2man properly underline them.
I removed them.
https://github.com/brianb/mdbtools/commit/9ee13359402483d570446bef904f0a7f888d6d44

------------------------------------------------------------------------------
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d

Commited doc changes

Nirgal — 2012-02-02T08:33:31

I just commited some small changes in documentation:
- See also sections
- mdb-header exit code

------------------------------------------------------------------------------
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d

Re: mdb-export with -I and -N

George L. Emigh — 2012-01-30T22:54:00

https://github.com/brianb/mdbtools/commit/7e34078bcc9a7b23ab028db969cdf77c0b283752

Excellent, thanks.



That part was not and should never actually a big problem, I am an old guy 
who dislikes spaces in things like table and field names and prefer them 
lower case as well, so it was just an issue of preference, and I changed 
everything to my liking with a little php scripting and I'm moving forward 
on my project.

So it's obviously not necessary to have those things part of mdbtools.

Thanks again for a fantastic tool.


------------------------------------------------------------------------------

-N documentation in man pages

Nirgal — 2012-01-30T11:13:50

I just added schema prefixing option in man documentation of mdb-schema and mdb-export:

https://github.com/brianb/mdbtools/commit/ad87174f10d96ca1ab27cb61feb70b6d2db9b143

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2

mdb-export with -I and -N

Nirgal — 2012-01-30T10:59:01

Ops. My mistake. I just comited a fix regarding namespaces:

$ src/util/mdb-export -I mysql ~/Desktop/uuid.mdb table1I
INSERT INTO `table1` (`test2`, `test`) VALUES ("OK",NULL);
INSERT INTO `table1` (`test2`, `test`) VALUES ("OK",NULL);

$ src/util/mdb-export -I mysql -N qp ~/Desktop/uuid.mdb table1I
INSERT INTO `qp_table1` (`test2`, `test`) VALUES ("OK",NULL);
INSERT INTO `qp_table1` (`test2`, `test`) VALUES ("OK",NULL);

$ src/util/mdb-export -I postgres ~/Desktop/uuid.mdb table1
INSERT INTO "table1" ("test2", "test") VALUES ("OK",NULL);
INSERT INTO "table1" ("test2", "test") VALUES ("OK",NULL);

$ src/util/mdb-export -I postgres -N qp ~/Desktop/uuid.mdb table1
INSERT INTO "qp"."table1" ("test2", "test") VALUES ("OK",NULL);
INSERT INTO "qp"."table1" ("test2", "test") VALUES ("OK",NULL);

Patch is available at:
https://github.com/brianb/mdbtools/commit/7e34078bcc9a7b23ab028db969cdf77c0b283752


I don't plan to work on sanitizing again (but maybe someone else will).
Non-ascii tables results in duplicate names. For exemple, chinese names will create tables "___" "___". One need to add something to diferentiate them, such as a number:
"___1" "___2".
Then you need to store new names for handling foreign contraints, and so on.
Not trivial at all.
With the proper escape now working, I don't see the point.
Removing spaces from table names after mysql conversion seems a bit out of scope, in my opinion.

Cheers

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2

Re: double changed to float from older version of mdbtools

George L. Emigh — 2012-01-27T19:32:58

Thank you, it kinda freaked me out a bit, ran into another problem, wanted 
to add a prefix to the table names using the -N <namespace> option.

for -N qp mdb-schema gives:
DROP TABLE [qp_Customer Table];
CREATE TABLE [qp_Customer Table]

which is what I expected, but mdb-export -I mysql -N qp gives:
INSERT INTO qp`Customer Table` (

Namespaces is not mentioned in the man pages but is when you run the command 
without any options, is Namespaces going to be removed?

Also I see the sanitize option is gone, will that option to convert spaces 
to underscores be coming back?  I liked the feature, convert all fields and 
table names to lowercase would be nice too, but for now I'll just alter my 
tables after the conversion.

Thank you so very much for the work you have done making this cool tool.

George



Jakob Egger wrote:

------------------------------------------------------------------------------
------------------------------------------------------------------------------

Re: double changed to float from older version of mdbtools

Jakob Egger — 2012-01-27T07:10:39

Hi George,

this is just a formating issue. The two numbers are just different ways of writing "15". When you insert "1.5000000000000000e+01" into a numeric field in a MySQL database, MySQL will store the value 15.

Best regards,
Jakob

On 26.01.2012, at 23:58, George L. Emigh wrote:



------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2

double changed to float from older version of mdbtools

George L. Emigh — 2012-01-26T22:58:04

I apologize if this is not the correct place to ask about this.

I am converting my data to MySQL.

I was using mdbtools-0.6_pre2 but had some issues that 0.7~rc1 seemed to 
resolve, I was able to build it fine, but one of the problems I have come 
across is stopping me.

in the 0.6 version I have a database that mdb-schema reports the Quantity 
field as:
Quantity Double

where 0.7 reports it as
Quantity  Float

In 0.6 the value reported by mdb-export is correct, in this example 15.
in 0.7 the value is reported as 1.5000000000000000e+01

Is this a bug, or should I just be using 0.6 and working around it's 
limitations.

Thank you for your time.

Re: autogen error compiling current version

Nirgal — 2012-01-22T09:06:53

Yes, I suspect libtool comments are outdated for freedsb. If you have any sugestion of improvement, please share with us. For exemple, I guess aclocal.m4 contains mdbtools specific macros that are required on other platforms.

Regarding glib2, I'm a real beginer with autoconf/automake/libtool and the pkgtools. So do take my opinion with care. I suspect my patch is not enough if you have glib2 installed, but not glib2-dev. So, we should find a way to detect it, or at least documented it in INSTALL exactly as you suggested.


On Saturday 21 January 2012 20:43:58 Dale Scott wrote:

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2

Re: autogen error compiling current version

Dale Scott — 2012-01-21T20:43:58

I'm OK with INSTALL as-is. For some reason, I mistakenly thought it listed more than libtool, automake, and autoconf (which it doesn't). It is worth proposing a patch for the comment regarding libtool on FreeBSD? (FreeBSD 8.2 and FreeBSD 9.0 both use libtool-2.4).




------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2

Re: autogen error compiling current version

Nirgal — 2012-01-21T19:36:57

I kinda disagree:

INSTALL documents the basic requirement for the build process to start:
    libtool
    automake
    autoconf

But there are many more dependencies such as:
bison
docbook
flex
libglade2-dev
libncurses-dev
openjade
txt2man
unixodbc-dev
...

I think these should be detected by configure.

After commit 5269dbfd7a4ec82af1830cc88a35230638902d99, I think you will get this error message at configure time:
glib 2.0 is required by MDB Tools (runtime and devel).
It can be downloaded at www.gtk.org.

Am I mistaken?

If you still think we should change INSTALL, please send a patch against
https://github.com/brianb/mdbtools/blob/master/INSTALL
There are "raw" and "edit this file" links on that page.

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2

Re: autogen error compiling current version

Dale Scott — 2012-01-20T14:27:52

Could I humbly suggest also adding glib2 to the list of required dependencies in the INSTALL file?

(I've heard I can submit the change myself all through the Github web interface, but not sure exactly how. If you gave me a general idea.......)




------------------------------------------------------------------------------
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d

Re: autogen error compiling current version

Nirgal — 2012-01-19T23:50:41

Hello

I think the issue is now fixed in master:
https://github.com/brianb/mdbtools/commit/5269dbfd7a4ec82af1830cc88a35230638902d99

Thank you for the bug report.

doc/Makefile.am & INSTALL

Nirgal — 2012-01-19T21:57:37

Commited

Also, I patched the sources origin in INSTALL file.

------------------------------------------------------------------------------
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d

Re: autogen error compiling current version

Dale Scott — 2012-01-16T20:04:45

Thanks Chris. On my FreeBSD-9.0 server, installing the glib2 port (v2.28.8) solved the problem.



Dale



----- Original Message -----
From: Chris Craig <ccraig< at >fibernetics.ca>
Date: Monday, January 16, 2012 8:40
Subject: Re: [mdb-dev] autogen error compiling current version
To: mdbtools-dev< at >lists.sourceforge.net


-----
Transparency with Trust
http://www.dalescott.net



------------------------------------------------------------------------------
RSA(R) Conference 2012
Mar 27 - Feb 2
Save $400 by Jan. 27
Register now!
http://p.sf.net/sfu/rsa-sfdev2dev2_______________________________________________
mdbtools-dev mailing list
mdbtools-dev< at >lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mdbtools-dev

Re: autogen error compiling current version

Chris Craig — 2012-01-16T15:00:12

...

I think this issue has been covered before, I imagine it's the same thing:

https://github.com/brianb/mdbtools/issues/6

autogen error compiling current version

Dale Scott — 2012-01-16T14:45:57

Hi, I tried compiling the current github head and get the following error in autogen (Linux Mint 11, autoconf 2.5):

dale< at >dale-ThinkPad-T61 ~/Projects/mdbtools $ ./autogen.sh
**Warning**: I am going to run `configure' with no arguments.
If you wish to pass any to it, please specify them on the
`./autogen.sh' command line.

processing .
Running aclocal ...
configure.in:117: warning: macro `AM_PATH_GLIB_2_0' not found in library
Running libtoolize...
libtoolize: putting auxiliary files in `.'.
libtoolize: copying file `./ltmain.sh'
libtoolize: Consider adding `AC_CONFIG_MACRO_DIR([m4])' to configure.in and
libtoolize: rerunning libtoolize, to keep the correct libtool macros in-tree.
libtoolize: Consider adding `-I m4' to ACLOCAL_AMFLAGS in Makefile.am.
Running autoheader...
Running automake --gnu ...
Running autoconf ...
Running ./configure ...
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
<snip>
checking for iconv... yeschecking for iconv declaration... extern size_t iconv (iconv_t cd, char * *inbuf, size_t *inbytesleft, char * *outbuf, size_t *outbytesleft);checking Are we using flex ... no - SQL engine disable./configure: line 11131: syntax error near unexpected token `2.0.0'./configure: line 11131: `AM_PATH_GLIB_2_0(2.0.0)'

I get the same error on FreeBSD 9.0, except the error message also notes there is an unexpected ")"

This didn't happen with the github head as of May 2010, but that was also on FreeBSD 8.2. I'm going to try re-compiling the old version again just to check what happens, but can anyone offer suggestions, or point out anything stupid I'm doing or not doing?

Thanks,
Dale

------------------------------------------------------------------------------
RSA(R) Conference 2012
Mar 27 - Feb 2
Save $400 by Jan. 27
Register now!
http://p.sf.net/sfu/rsa-sfdev2dev2_______________________________________________
mdbtools-dev mailing list
mdbtools-dev< at >lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mdbtools-dev

2 more patches

Nirgal — 2012-01-16T02:21:54

Attached are two very old patches that were hanging in Debian distro...

* treeviewmodel slightly improves the typing, so that less casts are necessary.

* filelocals reduce the scope of some local functions declaring them static.

These are small changes that I guess can go in. What do you think of these?
Description: Aestetic type changes
 In gmdb2/sql.c there are some incorrect uses of treeviews and models.
 Attached patch fixes or improves variable type definitions and adds and
 removes some typecasts as needed.
Author: Filip Van Raemdonck <mechanix< at >debian.org>
Forwarded: http://sourceforge.net/tracker/?func=detail&aid=1081706&group_id=2294&atid=302294
Reviewed-by: Jean-Michel Vourgère <jmv_deb< at >nirgal.com>
Last-Update: 2012-01-11

Index: mdbtools/src/gmdb2/sql.c
===================================================================
--- mdbtools.orig/src/gmdb2/sql.c
+++ mdbtools/src/gmdb2/sql.c
< at >< at > -55,6 +55,8 < at >< at >
 }
 }
 
+gchar* gmdb_export_get_filepath (GladeXML*);/* from table_export.c */
+
 /* callbacks */
 static void
 gmdb_sql_write_rslt_cb(GtkWidget *w, GladeXML *xml)
< at >< at > -74,7 +76,7 < at >< at >
 GtkWidget *treeview;
 GtkTreeViewColumn *col;
 GList *glist;
-GtkTreeStore *store;
+GtkTreeModel *store;
 GtkTreeIter iter;
 GValue value = { 0, };
 
< at >< at > -118,17 +120,16 < at >< at >
 g_list_free(glist);
 }
 
-store = (GtkTreeStore *) gtk_tree_view_get_model(GTK_TREE_VIEW(treeview));
-gtk_tree_model_get_iter_first(GTK_TREE_MODEL(store), &iter);
+store = gtk_tree_view_get_model(GTK_TREE_VIEW(treeview));
+gtk_tree_model_get_iter_first(store, &iter);
 rows=0;
 g_value_init (&value, G_TYPE_STRING);
 do {
 rows++;
-n_columns = gtk_tree_model_get_n_columns(GTK_TREE_MODEL(store));
+n_columns = gtk_tree_model_get_n_columns(store);
 for (i=0; i < n_columns; i++) {
 if (i>0) fputs(delimiter, outfile);
-gtk_tree_model_get_value(GTK_TREE_MODEL(store), 
-&iter, i, &value);
+gtk_tree_model_get_value(store, &iter, i, &value);
 str = (gchar *) g_value_get_string(&value);
 gmdb_print_quote(outfile, need_quote, quotechar, delimiter, str);
 fputs(str, outfile);
< at >< at > -136,7 +137,7 < at >< at >
 g_value_unset(&value);
 }
 fputs(lineterm, outfile);
-} while (gtk_tree_model_iter_next(GTK_TREE_MODEL(store), &iter));
+} while (gtk_tree_model_iter_next(store, &iter));
 
 fclose(outfile);
 gtk_widget_destroy(filesel);
< at >< at > -353,7 +354,8 < at >< at >
 MdbSQLColumn *sqlcol;
 GtkTextBuffer *txtbuffer;
 GtkTextIter start, end;
-GtkWidget *textview, *combo, *treeview, *store;
+GtkWidget *textview, *combo, *treeview;
+GtkTreeModel *store;
 /*GtkWidget *window;*/
 GList *history;
 GType *gtypes;
< at >< at > -402,17 +404,17 < at >< at >
 for (i=0;i<sql->num_columns;i++) 
 gtypes[i]=G_TYPE_STRING;
 
-store = (GtkWidget *) gtk_tree_view_get_model(GTK_TREE_VIEW(treeview));
+store = gtk_tree_view_get_model(GTK_TREE_VIEW(treeview));
 if (store) {
 while ((column = gtk_tree_view_get_column(GTK_TREE_VIEW(treeview), 0))) {
 gtk_tree_view_remove_column(GTK_TREE_VIEW(treeview), column);
 }
-gtk_widget_destroy(store);
+g_object_unref(store);
 }
-store = (GtkWidget *) gtk_list_store_newv(sql->num_columns, gtypes);
+store = (GtkTreeModel*)gtk_list_store_newv(sql->num_columns, gtypes);
 g_free(gtypes);
 
-gtk_tree_view_set_model(GTK_TREE_VIEW(treeview), GTK_TREE_MODEL(store));
+gtk_tree_view_set_model(GTK_TREE_VIEW(treeview), store);
 
 GtkCellRenderer *renderer;
 renderer = gtk_cell_renderer_text_new(); 
Description: Reduce function scopes
 This patch removes a few function declarations from the catch-all gmdb.h as
 they are defined and used only in a single file, and turns them in to static
 (file local) functions instead.
Author: Filip Van Raemdonck <mechanix< at >debian.org>
Forwarded: http://sourceforge.net/tracker/?func=detail&aid=1081716&group_id=2294&atid=302294
Reviewed-by: Jean-Michel Vourgère <jmv_deb< at >nirgal.com>
Last-Update: 2012-01-11

Index: mdbtools/src/gmdb2/gmdb.h
===================================================================
--- mdbtools.orig/src/gmdb2/gmdb.h
+++ mdbtools/src/gmdb2/gmdb.h
< at >< at > -57,10 +57,8 < at >< at >
 void gmdb_debug_close_all(void);
 
 /* sql.c */
-void gmdb_sql_save_as_cb(GtkWidget *w, GladeXML *xml);
 void gmdb_sql_new_cb(GtkWidget *w, gpointer data);
 void gmdb_sql_close_all(void);
-void gmdb_sql_save_query(GladeXML *xml, gchar *file_path);
 
 unsigned long gmdb_prefs_get_maxrows(void);
 
Index: mdbtools/src/gmdb2/sql.c
===================================================================
--- mdbtools.orig/src/gmdb2/sql.c
+++ mdbtools/src/gmdb2/sql.c
< at >< at > -39,8 +39,10 < at >< at >
 extern MdbHandle *mdb;
 extern MdbSQL *sql;
 
-static void gmdb_sql_tree_populate(MdbHandle *mdb, GladeXML *xml);
-static void gmdb_sql_load_query(GladeXML *xml, gchar *file_path);
+static void gmdb_sql_tree_populate (MdbHandle*, GladeXML*);
+static void gmdb_sql_load_query (GladeXML*, gchar*);
+static void gmdb_sql_save_query (GladeXML*, gchar*);
+static void gmdb_sql_save_as_cb (GtkWidget*, GladeXML*);
 
 void
 gmdb_sql_close_all()
< at >< at > -190,7 +192,7 < at >< at >
 }
 gmdb_sql_save_query(xml, str);
 }
-void
+static void
 gmdb_sql_save_as_cb(GtkWidget *w, GladeXML *xml)
 {
 GtkWindow *parent_window = (GtkWindow *) glade_xml_get_widget (xml, "gmdb");
< at >< at > -458,8 +460,7 < at >< at >
 }
 
 void
-gmdb_sql_new_cb(GtkWidget *w, gpointer data)
-{
+gmdb_sql_new_cb (GtkWidget *w, gpointer data) {
 GtkTargetEntry src;
 GtkWidget *mi, *but, *combo;
 GladeXML *sqlwin_xml;
< at >< at > -619,9 +620,9 < at >< at >
 textview = glade_xml_get_widget(xml, "sql_textview");
 g_object_set_data(G_OBJECT(textview), "file_name", file_name);
 }
-void
-gmdb_sql_save_query(GladeXML *xml, gchar *file_path)
-{
+
+static void
+gmdb_sql_save_query (GladeXML *xml, gchar *file_path) {
 FILE *out;
 GtkWidget *textview;
         GtkTextBuffer *txtbuffer;
< at >< at > -697,8 +698,7 < at >< at >
 #else
 
 void
-gmdb_sql_new_cb(GtkWidget *w, gpointer data)
-{
+gmdb_sql_new_cb (GtkWidget *w, gpointer data) {
 GtkWidget* dlg = gtk_message_dialog_new (gtk_widget_get_toplevel (w),
     GTK_DIALOG_DESTROY_WITH_PARENT, GTK_MESSAGE_INFO, GTK_BUTTONS_CLOSE,
     _("SQL support was not built.\nRun configure with the --enable-sql option."));
------------------------------------------------------------------------------
RSA(R) Conference 2012
Mar 27 - Feb 2
Save $400 by Jan. 27
Register now!
http://p.sf.net/sfu/rsa-sfdev2dev2_______________________________________________
mdbtools-dev mailing list
mdbtools-dev< at >lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mdbtools-dev