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

GNOME updates staging #60906

Merged
merged 5 commits into from May 7, 2019
Merged

Conversation

worldofpeace
Copy link
Contributor

Motivation for this change

Thing that probably needs most review here is ce4645f (folks uses meson!)

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS (not in this branch)
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@worldofpeace
Copy link
Contributor Author

Wait tracker doesn't belong in this pr 🤣

@@ -141,6 +141,7 @@ _multioutDocs() {
moveToOutput share/doc "${!outputDoc}"
moveToOutput share/gtk-doc "${!outputDevdoc}"
moveToOutput share/devhelp/books "${!outputDevdoc}"
moveToOutput share/devhelp/references "${!outputDevdoc}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you have a reference for this path? It does not seem to be used by Devhelp:

https://gitlab.gnome.org/GNOME/devhelp/blob/3.32.0/devhelp/dh-book-list-builder.c#L160-161

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't and it really kinda confused me because folks installs there so I assumed, even though I read the devhelp docs and it wasn't there, that perhaps it would use it?

Now that I think about it, it's probably a bug in folks. (also their gtk-docs stuff doesn't seem to be working)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

opened https://gitlab.gnome.org/GNOME/folks/issues/108. I'll just disable them in folks until it gets figured out.

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

Successfully merging this pull request may close these issues.

None yet

2 participants