Skip to content

php-sqlsrv, php-pdo_sqlsrv, msodbcsql: updates ; new Port - #33889

Open
BjarneDMat wants to merge 3 commits into
macports:masterfrom
BjarneDMat:php-sqlsrv
Open

php-sqlsrv, php-pdo_sqlsrv, msodbcsql: updates ; new Port#33889
BjarneDMat wants to merge 3 commits into
macports:masterfrom
BjarneDMat:php-sqlsrv

Conversation

@BjarneDMat

Copy link
Copy Markdown
Contributor

Description

  1. msodbcsql: add versions
    the provious update only had the newest version for macos_version_major >= 14
    this adds versions for all macos_version_major
  2. php-pdo_sqlsrv: new Port
    missing dependecy to php-sqlsrv
  3. php-sqlsrv: update to 5.13.2
    switched to use Github due to PECL being deprecated

@macportsbot

Copy link
Copy Markdown

Notifying maintainers:
@roederja for port msodbcsql.
@ryandesign for port php-sqlsrv.

Comment thread php/php-pdo_sqlsrv/Portfile Outdated
Comment thread php/php-pdo_sqlsrv/Portfile Outdated
Comment thread php/php-pdo_sqlsrv/Portfile Outdated
Comment thread php/php-pdo_sqlsrv/Portfile Outdated
Comment thread php/php-sqlsrv/Portfile
Comment on lines -38 to +42
version 5.12.0
github.setup microsoft msphpsql 5.12.0 v
revision 1
checksums rmd160 6b307e55af04354f862eb805ed5761287683bce4 \
sha256 a9ebb880b2a558d3d6684f6e6802c53c5bffa49e1ee60d1473a7124fc9cb72ad \
size 193767
checksums rmd160 d741e123d82bdc85a4c40ad27a5ec272e74e60e9 \
sha256 c2a67f20a49555243a1b6c45d4a735e58dcbed0bafefbf7de8e196f317544fbc \
size 10776978

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this is a stealth update; happens more often below. You do bot update the version but do change the location and thus the checksums. This will result in failures because the mirror distfiles are different.

@BjarneDMat BjarneDMat Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

🤔 so, I'll need to +1 the revisions ... done ‼️
I've taken note of this ... this will be a recuring thing as I gradually transition from PECL

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No, that is incorrect; see the relevant part in the Wiki

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@reneeotten
🤓❓ Do I put the dist_subdir ${name}/${version}_1 in ...

  1. the if {${subport} ne ${name}} { ... } and then delete it at the next update
  2. each elseif {[vercmp ${php.branch} >= <phpver>]} { ... } and then delete them at the next update
  3. each elseif {[vercmp ${php.branch} >= <phpver>]} { ... } and then never delete them

The explanation in the wiki really only covers it if there're no subports

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

5 participants