it is impossible to install with mysqli on a non standard port. it is magical how to do so with mysql and postgres (append : in the host textfield). i suggest making a UI in the 'advanced fieldset' for port number.

CommentFileSizeAuthor
#7 patch_122_2.patch6.7 KBflk
#5 patch_122_1.patch6.81 KBflk
#4 patch_122_0.patch6.81 KBflk
#1 patch_1226.7 KBmoshe weitzman

Comments

moshe weitzman’s picture

moshe weitzman commented
Status: Active » Needs review
StatusFileSize
new patch_1226.7 KB
flk’s picture

flk commented

patch works fine for me: tried 3 different ports
9000
3309
4086

defualt port 3306 and leaving it blank.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

RTBC based on prior post

flk’s picture

flk commented
StatusFileSize
new patch_122_0.patch6.81 KB

corrected wrong tabbing..thnx webchick

flk’s picture

flk commented
StatusFileSize
new patch_122_1.patch6.81 KB

and another

*holds his breath*

AjK’s picture

AjK commented

Patch in #5 failed to apply to a fresh CVS install (all hunks failed).

Patch in #1 applied no problems. Verify that all worked fine with #1 patch as specified in the problem/solution.

regards,
--AjK

flk’s picture

flk commented
StatusFileSize
new patch_122_2.patch6.7 KB

pardon my noobishness, #1 worked perfectly fine for me aswell, i was just told to correct 2 sytanx which were tabs instead of tabs.

hope i corrected it now

flk’s picture

flk commented

s/of tabs/of space

AjK’s picture

AjK commented

Patch in #7 applied and works as expected (install for mysqli functional).

regards,
--AjK

dries’s picture

dries commented
Status: Reviewed & tested by the community » Needs work

Patch no longer applies. Otherwise looks OK.

AjK’s picture

AjK commented

Dries, I'm confused. I got myself a fresh checkout of HEAD and re-applied the patch in #7 above:-

ajk@holly:~/WS/DRUPAL/DRUPAL-5-0-0-RCx1/drupal$ patch -p0 > patch_122_2.patch
Hmm... Looks like a unified diff to me...
The text leading up to this was:
--------------------------
|=== modified file 'includes/install.mysqli.inc'
|--- includes/install.mysqli.inc 2006-08-20 06:38:50 +0000
|+++ includes/install.mysqli.inc 2006-09-04 00:59:58 +0000
--------------------------
Patching file includes/install.mysqli.inc using Plan A...
Hunk #1 succeeded at 33.
Hmm... The next patch looks like a unified diff to me...
The text leading up to this was:
--------------------------
|
|=== modified file 'install.php'
|--- install.php 2006-09-01 08:44:53 +0000
|+++ install.php 2006-09-04 00:54:01 +0000
--------------------------
Patching file install.php using Plan A...
Hunk #1 succeeded at 109.
Hunk #2 succeeded at 131.
Hunk #3 succeeded at 153.
Hunk #4 succeeded at 163.
Hunk #5 succeeded at 250.
Hunk #6 succeeded at 290.
Hunk #7 succeeded at 309.
Hunk #8 succeeded at 323.
Hmm... Ignoring the trailing garbage.
done

Seems to apply ok to me?

regards,
--AjK

dries’s picture

dries commented
Status: Needs work » Fixed

You're right. Committed to CVS HEAD.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)