-
Notifications
You must be signed in to change notification settings - Fork 46
Record Safari Technology Preview version #617
Comments
Another failed attempt:
@burg, how would you advise people to get the STP release number if it wasn't installed from a known dmg? (Which I am currently doing, I believe @jugglinmike has told you that safaridriver doesn't work in STP 66, hence the 65 in this issue.) |
@burg https://bugreport.apple.com/web/?problemID=43691655 is filed on the inability to get the STP release number programmatically FWIW |
Safari Technology Preview 67 and later has added the --version flag to safaridriver. Let me know if that doesn't solve your needs. |
Wonderful, thanks @burg! |
I don't see 67 on https://developer.apple.com/safari/download/ yet, do you know when it will be released, @burg? Thanks! |
You could also check the WebKit blog. Looks like version 67 hasn't been announced yet. Also, Brian: the ATOM feed and the RSS feed are both out of date (they list the announcement of STP 62 as the latest post). |
It's here: https://webkit.org/blog/8419/release-notes-for-safari-technology-preview-67/ I've sent Homebrew/homebrew-cask-versions#6387 for my own use, and verified that
|
@burg, do you also know what that string will be in Safari stable? I'm asking because I'm updating |
Example: bburg$ safaridriver --version |
The version string has been explicitly designed to be regex parsable. Let me know if you think this is problematic; I've used it internally with pytest to write Safari Technology Preview-specific tests. |
Thanks @burg! web-platform-tests/wpt#13467 is where I'm writing the code to parse it, although I haven't finished it. BTW, 12.1 doesn't show up in the string for STP, but it is the version that PlistBuddy will report. Do you consider part of the version, or just an internal detail? |
12.1 is also in the UI screenshot I posted. |
Safari Technology Preview's Info.plist will list the next anticipated Safari release version, to signal that it is not Safari 12 (in this case). However I omit that from the version output as we typically don't reference that version string for Safari Technology Preview. There is no other programmatic way to determine the STP release from the Safari Technology Preview app. |
Thanks for the explanation, then so think just stripping "Included with " in both cases to get "Safari Technology Preview (Release 67, 13607.1.9.0.1)" and "Safari 12.1 (14607.1.11)" is fine. @jugglinmike, would you know why 12.1 shows up as the version for STP on wpt.fyi now? Did you use PlistBuddy? @lukebjerring, will we need to adapt the wpt.fyi frontend to this as well? And can we change the version information retroactively to not have 12.1 ever match STP? |
When running Safari locally, the system infers the version based on files that are "near" the executable under test. The heuristic is implemented and explained in the
You might say we're our own PlistBuddy |
Aha, well that explains it at least. And means that when wpt can get the version from @jugglinmike, do you know which exact STP versions have been used for which runs, so we could retroactively edit the version info? |
We've tested the latest version for almost every collection. The exception is we tested STP 65 until STP 67 was released because version 66 could not be automated. Relevant data is available in the build logs, so I can verify programtically when/if that's desirable. |
Great, thank you! @lukebjerring, do you think we should just edit the version information in wpt.fyi's datastore, or will something bad happen if the information is out of sync with the reports? We could just leave it, but downside is that |
Yes; I'll defer to @Hexcles for a batch-update of mapping the (incorrect) data |
The `--version` argument is supported starting in Safari 12.1: web-platform-tests/results-collection#617
…18454) The `--version` argument is supported starting in Safari 12.1: web-platform-tests/results-collection#617
…eb-platform-tests#18454) The `--version` argument is supported starting in Safari 12.1: web-platform-tests/results-collection#617
…r `safaridriver --version`, a=testonly Automatic update from web-platform-tests [Azure Pipelines] drop old workaround for `safaridriver --version` (#18454) The `--version` argument is supported starting in Safari 12.1: web-platform-tests/results-collection#617 -- wpt-commits: a2d1acf3d4e8fe76eba70ee9b5f69dba2189819f wpt-pr: 18454
…r `safaridriver --version`, a=testonly Automatic update from web-platform-tests [Azure Pipelines] drop old workaround for `safaridriver --version` (#18454) The `--version` argument is supported starting in Safari 12.1: web-platform-tests/results-collection#617 -- wpt-commits: a2d1acf3d4e8fe76eba70ee9b5f69dba2189819f wpt-pr: 18454
…r `safaridriver --version`, a=testonly Automatic update from web-platform-tests [Azure Pipelines] drop old workaround for `safaridriver --version` (#18454) The `--version` argument is supported starting in Safari 12.1: web-platform-tests/results-collection#617 -- wpt-commits: a2d1acf3d4e8fe76eba70ee9b5f69dba2189819f wpt-pr: 18454 UltraBlame original commit: 5bdb6f1e8f37dfdc0ca96fdb8ae79ea0a36f2483
…r `safaridriver --version`, a=testonly Automatic update from web-platform-tests [Azure Pipelines] drop old workaround for `safaridriver --version` (#18454) The `--version` argument is supported starting in Safari 12.1: web-platform-tests/results-collection#617 -- wpt-commits: a2d1acf3d4e8fe76eba70ee9b5f69dba2189819f wpt-pr: 18454 UltraBlame original commit: 5bdb6f1e8f37dfdc0ca96fdb8ae79ea0a36f2483
…r `safaridriver --version`, a=testonly Automatic update from web-platform-tests [Azure Pipelines] drop old workaround for `safaridriver --version` (#18454) The `--version` argument is supported starting in Safari 12.1: web-platform-tests/results-collection#617 -- wpt-commits: a2d1acf3d4e8fe76eba70ee9b5f69dba2189819f wpt-pr: 18454 UltraBlame original commit: 5bdb6f1e8f37dfdc0ca96fdb8ae79ea0a36f2483
The raw report for the latest run has
"browser_version": "12.1"
. This is also what shows up on wpt.fyi.In the UI a much more detailed version string ("Release 65 (Safari 12.1, WebKit 13607.1.5.2)") is available:
Knowing the release number (65) and when it changes is important for comparing results over time.
Unfortunately,
/usr/libexec/PlistBuddy -c Print /Applications/Safari\ Technology\ Preview.app/Contents/Info.plist
doesn't include the "65", but it does include "13607.1.5.2" as CFBundleVersion.P.S. This issue is similar to web-platform-tests/wpt#13052 and web-platform-tests/wpt#13399, and the solution may be in wpt itself, but also perhaps not given that much of the Safari-specific setup is in this repo.
The text was updated successfully, but these errors were encountered: