Skip to content

Conversation

GuillaumeGomez
Copy link
Member

Fixes this issue:

error: couldn't generate documentation: failed to read column from disk: data consumer error: missing field `unknown number` at line 1 column 8
  |
  = note: failed to create or modify "build/x86_64-unknown-linux-gnu/test/rustdoc-gui/doc/search.index/entry/": failed to read column from disk: data consumer error: missing field `unknown number` at line 1 column 8

warning: `theme_css` (lib doc) generated 1 warning
error: could not document `theme_css`

The problem was that a conversion was forgotten for the ItemType enum.

Thanks a lot to @janis-bhm!

r? @lolbinarycat

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Sep 2, 2025
@GuillaumeGomez
Copy link
Member Author

@bors r=notriddle rollup

@bors
Copy link
Collaborator

bors commented Sep 2, 2025

📌 Commit ad2e096 has been approved by notriddle

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 2, 2025
bors added a commit that referenced this pull request Sep 2, 2025
Rollup of 14 pull requests

Successful merges:

 - #144066 (stabilize c-style varargs for sysv64, win64, efiapi, aapcs)
 - #145783 (add span to struct pattern rest (..))
 - #146034 (Update target spec metadata of Arm64EC Windows and Trusty targets)
 - #146064 (Add compiler error when trying to use concat metavar expr in repetitions)
 - #146070 (rustdoc-search: skip loading unneeded fnData)
 - #146088 (constify impl Try for ControlFlow)
 - #146089 (fix a constness ordering bug in rustfmt)
 - #146091 (fix rustdoc `render_call_locations`  panicking because of default span `DUMMY_SP` pointing at non local-source file)
 - #146094 (Make `Parser::parse_for_head` public for rustfmt usage)
 - #146102 (Remove dead code stemming from an old effects desugaring)
 - #146115 (Add maintainer for VxWorks)
 - #146116 (Adjust issue-118306.rs test after LLVM change)
 - #146117 (Fix search index generation)
 - #146118 (improve process::abort rendering in Miri backtraces)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 0e203c4 into rust-lang:master Sep 2, 2025
10 checks passed
@rustbot rustbot added this to the 1.91.0 milestone Sep 2, 2025
rust-timer added a commit that referenced this pull request Sep 2, 2025
Rollup merge of #146117 - GuillaumeGomez:fix-search-index-generation, r=notriddle

Fix search index generation

Fixes this issue:

```
error: couldn't generate documentation: failed to read column from disk: data consumer error: missing field `unknown number` at line 1 column 8
  |
  = note: failed to create or modify "build/x86_64-unknown-linux-gnu/test/rustdoc-gui/doc/search.index/entry/": failed to read column from disk: data consumer error: missing field `unknown number` at line 1 column 8

warning: `theme_css` (lib doc) generated 1 warning
error: could not document `theme_css`
```

The problem was that a conversion was forgotten for the `ItemType` enum.

Thanks a lot to `@janis-bhm!`

r? `@lolbinarycat`
@GuillaumeGomez GuillaumeGomez deleted the fix-search-index-generation branch September 2, 2025 21:16
tgross35 added a commit to tgross35/rust that referenced this pull request Sep 3, 2025
…type-test, r=GuillaumeGomez

rustdoc-search: add test case for indexing every item type

Test case for rust-lang#146117
tgross35 added a commit to tgross35/rust that referenced this pull request Sep 3, 2025
…type-test, r=GuillaumeGomez

rustdoc-search: add test case for indexing every item type

Test case for rust-lang#146117
github-actions bot pushed a commit to rust-lang/miri that referenced this pull request Sep 3, 2025
Rollup of 14 pull requests

Successful merges:

 - rust-lang/rust#144066 (stabilize c-style varargs for sysv64, win64, efiapi, aapcs)
 - rust-lang/rust#145783 (add span to struct pattern rest (..))
 - rust-lang/rust#146034 (Update target spec metadata of Arm64EC Windows and Trusty targets)
 - rust-lang/rust#146064 (Add compiler error when trying to use concat metavar expr in repetitions)
 - rust-lang/rust#146070 (rustdoc-search: skip loading unneeded fnData)
 - rust-lang/rust#146088 (constify impl Try for ControlFlow)
 - rust-lang/rust#146089 (fix a constness ordering bug in rustfmt)
 - rust-lang/rust#146091 (fix rustdoc `render_call_locations`  panicking because of default span `DUMMY_SP` pointing at non local-source file)
 - rust-lang/rust#146094 (Make `Parser::parse_for_head` public for rustfmt usage)
 - rust-lang/rust#146102 (Remove dead code stemming from an old effects desugaring)
 - rust-lang/rust#146115 (Add maintainer for VxWorks)
 - rust-lang/rust#146116 (Adjust issue-118306.rs test after LLVM change)
 - rust-lang/rust#146117 (Fix search index generation)
 - rust-lang/rust#146118 (improve process::abort rendering in Miri backtraces)

r? `@ghost`
`@rustbot` modify labels: rollup
tgross35 added a commit to tgross35/rust that referenced this pull request Sep 3, 2025
…type-test, r=GuillaumeGomez

rustdoc-search: add test case for indexing every item type

Test case for rust-lang#146117
jhpratt added a commit to jhpratt/rust that referenced this pull request Sep 3, 2025
…type-test, r=

rustdoc-search: add test case for indexing every item type

Test case for rust-lang#146117
Zalathar added a commit to Zalathar/rust that referenced this pull request Sep 4, 2025
…type-test, r=GuillaumeGomez

rustdoc-search: add test case for indexing every item type

Test case for rust-lang#146117
rust-timer added a commit that referenced this pull request Sep 4, 2025
Rollup merge of #146131 - notriddle:rustdoc-search-load-itemtype-test, r=GuillaumeGomez

rustdoc-search: add test case for indexing every item type

Test case for #146117
github-actions bot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Sep 8, 2025
…, r=GuillaumeGomez

rustdoc-search: add test case for indexing every item type

Test case for rust-lang/rust#146117
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants