Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

geckodriver 0.26.0 #45217

Closed
wants to merge 1 commit into from
Closed

Conversation

chenrui333
Copy link
Member

Created with brew bump-formula-pr.

@chenrui333
Copy link
Member Author

install step failed

23:28:45 ==> cargo install --root /usr/local/Cellar/geckodriver/0.26.0 --path .
23:28:45 error: `/private/tmp/geckodriver-20191012-64895-17lpif8/geckodriver-0.26.0` does not contain a Cargo.toml file. --path must point to a directory containing a Cargo.toml file.

@chenrui333 chenrui333 added the build failure CI fails while building the software label Oct 12, 2019
@chenrui333
Copy link
Member Author

https://github.com/mozilla/geckodriver/releases/tag/v0.26.0

Note to macOS Catalina users: Due to the recent requirement
from Apple that all programs be notarized, geckodriver will
not work on Catalina. We are working on a fix for this
and you can follow the progress on this in bug 1588081.

Note also that with this release the minimum recommended Firefox version
has changed to Firefox ≥60.

@chrmoritz
Copy link
Contributor

chrmoritz commented Oct 15, 2019

Looks like the GitHub tag for this release doesn't contain any source code at all (compare with the v0.25.0 tag). Might need to check out their hg repository for commit e9783a644016aa9b317887076618425586730d73 to get the actual source code for this release (similar to HEAD build).

@chenrui333
Copy link
Member Author

Yeah, that is interesting comment, let me switch to use git-tag thing.

@chrmoritz
Copy link
Contributor

chrmoritz commented Oct 15, 2019

No, I meant they didn't published the source code for 0.26.0 on GitHub at all. Previously they at least pushed the source code with the corresponding tags only. Not it looks like it's only available in their hg repository. Not sure if this was intended or not.

url "https://hg.mozilla.org/mozilla-central/", :using => :hg, :revision => "e9783a644016aa9b317887076618425586730d73"

cd "testing/geckodriver" # unconditionally

@chenrui333
Copy link
Member Author

chenrui333 commented Oct 15, 2019

ok, I will submit an issue in their repo to see how it goes.

@andreastt
Copy link

Hi, the maintainer of geckodriver here.

No, I meant they didn't published the source code for 0.26.0 on GitHub at all. Previously they at least pushed the source code with the corresponding tags only. Not it looks like it's only available in their hg repository. Not sure if this was intended or not.

Some Rust crate dependencies now exist only in source form inside mozilla-central, and their releases are not always coordinated with geckodriver. If geckodriver works with a pinned, previous crate version this is more incidental than intended. It used to be a best-effort process to put together something that built outside of the canonical source code repository, but this is no longer possible.

Because we publish release binaries that require secret signing keys (and soon notarisation) from the Firefox build/CI system, we can no longer guarantee that a source code export with pinned crate dependencies will make a reproducible build.

I was not aware geckodriver was released on Homebrew and I’m sorry for the trouble this causes you. Please let me know if there’s anything I can do or questions I can answer that will make it possible for you to resolve this breakage.

@andreastt
Copy link

I forgot to mention that once you have a mozilla-central checkout of the Mercurial SHA-1 mentioned in the release notes, you should be able to follow the build instructions documented here: https://firefox-source-docs.mozilla.org/testing/geckodriver/Building.html

You may need to run ./mach configure first. If that doesn’t work you may also need ./mach bootstrap to get all the right build dependencies.

@chenrui333
Copy link
Member Author

That sounds about right, I will close the PR for now.

@chenrui333 chenrui333 closed this Oct 22, 2019
@lock lock bot added the outdated PR was locked due to age label Jan 5, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Jan 5, 2020
@chenrui333 chenrui333 deleted the geckodriver-0.26.0 branch December 18, 2022 04:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
build failure CI fails while building the software outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants