Added an Audit step for FreeBSD and DragonFly packagees.

[wolpert]

Allows for auditing the packages to be disabled since they are breaking steps. Current behaivor is the default, where if the audit fails topgrade stops. Can be disabled in the [misc] section independenly from other sections.

Probably we should have a document explaining all the Steps in the system. Maybe as a separate PR?

This is a potential fix for the bug I reported here: #639

Standards checklist:

• The PR title is descriptive.
• I have read CONTRIBUTING.md
• The code compiles (cargo build)
• The code passes rustfmt (cargo fmt)
• The code passes clippy (cargo clippy)
• The code passes tests (cargo test)
• Optional: I have tested the code myself

For new steps

• Optional: Topgrade skips this step where needed
• Optional: The --dry-run option works with this step
• Optional: The --yes option works with this step if it is supported by
  the underlying command

If you developed a feature or a bug fix for someone else and you do not have the
means to test it, please tag this person here.

[wolpert]

BTW, tested in FreeBSD.

With default configuration file, there is no change in behavior.

When the following is added to the configuration file, the Audit step is removed.

[misc]
disable = ["audit"]

[SteveLauC]

Hi, thanks for this patch!

If you want to make audit a step, you should also call print_separator() in that function:

pub fn audit_packages(ctx: &ExecutionContext) -> Result<()> {
    let sudo = require_option(ctx.sudo().as_ref(), REQUIRE_SUDO.to_string())?;
    
    print_separator("FreeBSD Audit");
    
    Command::new(sudo)
        .args(["/usr/sbin/pkg", "audit", "-Fr"])
        .status_checked()
}

[wolpert]

If you want to make audit a step, you should also call print_separator() in that function:

Good idea. Just updated with the change to both FreeBSD and DragonFly. Thanks!

[SteveLauC]

Thanks!