Skip to content
  • Angelos Evripiotis's avatar
    userconfig: rm really-workspace-close-project-inaccessible · c24f2971
    Angelos Evripiotis authored and Angelos Evripiotis's avatar Angelos Evripiotis committed
    Remove the need for the 'really-workspace-close-project-inaccessible'
    config option, as well as the option itself.
    
    As agreed on the mailing list [1], all the 'are you sure?' prompts on
    workspace reset and close were removed. While that discussion was going
    on, this new prompt and option was added. At the 2019 BuildStream
    Gathering, it was verbally agreed between myself and Tristan VB that we
    would also remove this instance.
    
    It was also agreed that we should have a notice to let the user know
    what they'd done, this was already in place if interactive. Moved it to
    be unconditional so that there's no difference in non-interactive
    behaviour. Made it output to stderr, as it's diagnostic meant for the
    user. Made it the last thing echo'd so it's next to the prompt - it's
    very relevant to what they type next. Added a test to make sure the text
    makes it to stderr in the appropriate case, and not in an inappropriate
    one.
    
    This is the last instance of any prompt configuration, so BuildStream
    can also forget all of that machinery.
    
    [1] https://mail.gnome.org/archives/buildstream-list/2018-December/msg00111.html
    c24f2971