Skip to content

Conversation

@foxycode
Copy link
Contributor

@foxycode foxycode commented Jan 18, 2017

  • bug fix? yes
  • new feature? no
  • BC break? no

Still not working as expected, but I think it's appveyor fault (SQL servers don't start). In any case, it's at least some progress. I spent lot of time with this, so I hope it won't be useless.

@foxycode foxycode changed the title appveyor: upgrade php to version 7.0 appveyor: upgrade php to version 7.1 Jan 18, 2017
@foxycode foxycode changed the title appveyor: upgrade php to version 7.1 appveyor: upgrade php to version 7.0 Jan 18, 2017
@foxycode
Copy link
Contributor Author

Fixed as much as it could be. Returned to php 7.0 because compatible sqlsrv binaries for 7.1 aren't available.

@dg
Copy link
Member

dg commented Jan 18, 2017

https://github.com/Microsoft/msphpsql/releases/download/4.1.4-Windows/7.0.zip is better than master

@foxycode
Copy link
Contributor Author

Great, I missed that pre-release. It contains binaries for 7.1 too, so I could change to 7.1 again :)

Shouldn't I remove thos failing SQL Server istances?

@foxycode foxycode changed the title appveyor: upgrade php to version 7.0 appveyor: upgrade php to version 7.1 Jan 18, 2017
@dg dg merged commit a4f2cc0 into nette:master Jan 18, 2017
@foxycode foxycode deleted the appveyor branch January 18, 2017 23:39
dg pushed a commit that referenced this pull request Jan 24, 2017
dg pushed a commit that referenced this pull request Jan 24, 2017
dg pushed a commit that referenced this pull request Jan 25, 2017
dg pushed a commit that referenced this pull request Jan 25, 2017
dg pushed a commit that referenced this pull request Jan 25, 2017
dg pushed a commit that referenced this pull request Jan 25, 2017
dg pushed a commit that referenced this pull request Jan 25, 2017
dg pushed a commit that referenced this pull request Jan 26, 2017
dg pushed a commit that referenced this pull request Jan 26, 2017
dg pushed a commit that referenced this pull request Jan 29, 2017
dg pushed a commit that referenced this pull request Jan 29, 2017
dg pushed a commit that referenced this pull request Feb 2, 2017
appveyor.yml Outdated
- IF %PHP%==1 cd c:\php
- IF %PHP%==1 appveyor DownloadFile http://windows.php.net/downloads/releases/archives/php-5.6.14-Win32-VC11-x86.zip
- IF %PHP%==1 7z x php-5.6.14-Win32-VC11-x86.zip >nul
- IF %PHP%==1 appveyor DownloadFile http://windows.php.net/downloads/releases/php-7.0.15-Win32-VC14-x86.zip
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

appveyor.yml Outdated
- IF %PHP%==1 appveyor DownloadFile https://files.nette.org/misc/php-sqlsrv.zip
- IF %PHP%==1 7z x php-sqlsrv.zip >nul
- IF %PHP%==1 copy SQLSRV\php_pdo_sqlsrv_56_ts.dll ext\php_pdo_sqlsrv_ts.dll
- IF %PHP%==1 appveyor DownloadFile https://github.com/Microsoft/msphpsql/archive/master.zip
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is possible to use stable branch instead of master?

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