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.

Sign up for GitHub

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

Jump to bottom

[WIP] Update XML paths #30457

Closed
wants to merge 30 commits into from
Closed

[WIP] Update XML paths #30457

wants to merge 30 commits into from

Conversation

jtojnar
Copy link
Contributor

@jtojnar jtojnar commented Oct 16, 2017 •
edited

Depends on #30428.

Motivation for this change

The paths without share/ are deprecated.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • 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 nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@jtojnar jtojnar changed the title Update XML paths [WIP] Update XML paths Oct 16, 2017
@jtojnar jtojnar mentioned this pull request Oct 16, 2017
Closed
8 tasks
@jtojnar jtojnar force-pushed the xml-paths branch 6 times, most recently from 0a854c6 to c854334 Compare October 16, 2017 01:59
peterhoeg and others added 22 commits October 18, 2017 13:25
@peterhoeg @jtojnar
docbook: put DTDs in $out/share/xml instead of $out/xml
d433a96
@jtojnar
doc: Update XML paths
630cac4
@jtojnar
nixos manual: update XML paths
0a01803
@jtojnar
kdelibs4: update XML paths
e853026
@jtojnar
cvs_fast_export: rely on propagated findXMLCatalogs hook
f999ac6
@jtojnar
reposurgeon: rely on propagated findXMLCatalogs hook
535af41
@jtojnar
gnome3.rarian: remove unnecessary DocBook dependency
cf5e051
@jtojnar
gnome2.rarian: remove unnecessary DocBook dependency
7a4c313
@jtojnar
gnome2.scrollkeeper: update XML paths
c0d6506
@jtojnar
libsForQt5.kdelibs4support: update XML paths
5abf6f1
@jtojnar
kdoctools: update XML paths
e7d82c1
@jtojnar
cppcheck: update XML paths
da642ed
@jtojnar
ima-evm-utils: update XML paths
3fdfaef
@jtojnar
lxc: rely on propagated findXMLCatalogs hook
8f4518c
@jtojnar
nftables: rely on propagated findXMLCatalogs hook
7eb1210
@jtojnar
shadow: rely on propagated findXMLCatalogs hook
2cd5fad 
shadow uses a copy of m4 from gtk-doc so we need to apply the same
fix as in 407db7b.
@jtojnar
sssd: rely on propagated findXMLCatalogs hook
b1bf0c0
@jtojnar
trace-cmd: update XML paths
228505d
@jtojnar
irker: rely on propagated findXMLCatalogs hook
bd00ae4
@jtojnar
debian-devscripts: update XML paths
fd4db1c
@jtojnar
moreutils: update XML paths
47a3cd9
@jtojnar
nixops: update XML paths
736817d
@jtojnar jtojnar mentioned this pull request Nov 22, 2017
Draft
8 tasks
@mmahut
Copy link
Member

mmahut commented Aug 11, 2019

Are there any updates on this pull request, please?

@jtojnar
Copy link
Contributor Author

jtojnar commented Aug 11, 2019

Lot of this is now part of #31952.

@jtojnar jtojnar marked this pull request as draft June 1, 2020 05:22
@stale
Copy link

stale bot commented Jun 4, 2021

I marked this as stale due to inactivity. → More info

@stale stale bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jun 4, 2021
@Artturin Artturin closed this May 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adamcstephens adamcstephens Awaiting requested review from adamcstephens adamcstephens will be requested when the pull request is marked ready for review adamcstephens is a code owner

@K900 K900 Awaiting requested review from K900 K900 will be requested when the pull request is marked ready for review K900 is a code owner

@NickCao NickCao Awaiting requested review from NickCao NickCao will be requested when the pull request is marked ready for review NickCao is a code owner

@SuperSandro2000 SuperSandro2000 Awaiting requested review from SuperSandro2000 SuperSandro2000 will be requested when the pull request is marked ready for review SuperSandro2000 is a code owner

@ttuegel ttuegel Awaiting requested review from ttuegel ttuegel will be requested when the pull request is marked ready for review ttuegel is a code owner

@stigtsp stigtsp Awaiting requested review from stigtsp stigtsp will be requested when the pull request is marked ready for review stigtsp is a code owner

@zakame zakame Awaiting requested review from zakame zakame will be requested when the pull request is marked ready for review zakame is a code owner

@dasJ dasJ Awaiting requested review from dasJ dasJ will be requested when the pull request is marked ready for review dasJ is a code owner

Assignees
No one assigned
Labels
2.status: merge conflict 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md 2.status: work-in-progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@jtojnar @mmahut @ryantm @veprbl @Artturin @peterhoeg