[107049][Easy] Modify readme to suggest using configure

[lionellloh]

This is related to #107050. Modifying instructions in readme to use configure instead of printf

Manual Test:

Command

lionellloh@lionellloh-mbp rust % ./configure --set changelog-seen=1 --set profile=user                   

configure: processing command line
configure: 
configure: changelog-seen       := 1
configure: profile              := user
configure: build.configure-args := ['--set', 'changelog-seen=1', '--set', 'profil ...
configure: 
configure: writing `config.toml` in current directory
configure: 
configure: run `python /Users/lionellloh/rust/x.py --help`

Result

16c16
< changelog-seen = 1
---
> changelog-seen = 2
26c26
< profile = user
---
> #profile = <none>
331c331
< configure-args = ['--set', 'changelog-seen=1', '--set', 'profile=user']
---
> #configure-args = []
678c678
< [target.x86_64-apple-darwin]
---
> [target.x86_64-unknown-linux-gnu]
812d811
< 

[rustbot]

r? @Mark-Simulacrum

(rustbot has picked a reviewer for you, use r? to override)

[albertlarsan68]

I think this should be changed to suggest using ./x setup user

[lionellloh]

I think this should be changed to suggest using ./x setup user

Ooh do you mind explaining why so?

I am new so just passing the message from #107049 from @jyn514

We should make it possible to set top-level keys through configure, both changelog-seen and profile. We should also update the line in the readme linked above to suggest ./configure instead of printf.

[jyn514]

@albertlarsan68 they are not quite the same. x setup user requires building bootstrap first and sometimes distros want to apply configuration when building bootstrap itself.

[jyn514]

@bors r+ rollup

[bors]

📌 Commit dccd4ec has been approved by jyn514

It is now in the queue for this repository.

[jyn514]

@bors r-

[albertlarsan68]

Then I think there should be a section for the distros, and one for the users wanting to build from source.

[jyn514]

I don't really want to add more blessed paths 😓 bootstrap has enough configuration knobs as-is.

What are you hoping to gain from having people run x setup instead?

[jyn514]

Wait, so I read the original issue and this doesn't actually do what I want - I want ./configure to set profile=user by default, even if --set profile isn't passed.

[albertlarsan68]

In one of the guides, it is recommended that x is used rather than configure, since the makefile it produces is just a wrapper around x.

[lionellloh]

I think I have applied the change requested @jyn514

[jyn514]

Wait, so I read the original issue and this doesn't actually do what I want - I want ./configure to set profile=user by default, even if --set profile isn't passed.

@lionellloh I don't see this change.

[jyn514]

In one of the guides, it is recommended that x is used rather than configure, since the makefile it produces is just a wrapper around x.

Do you have a link to this? I vaguely remember some text saying "configure generates config.toml", but not a recommendation against using it.

[jyn514]

Hi @lionellloh, it's been a while - do you know when you'll have time to look at this again? Is it clear what to do?

[jyn514]

Going to close this in favor of #110541.