From 7aed615a8d65003b79393f5ffa39765e296576a7 Mon Sep 17 00:00:00 2001 From: Adam Williamson Date: Sat, 26 Jan 2019 20:49:59 +0100 Subject: [PATCH] Only do the addrepourl check if we're on F28+ inst.addrepo was added in F29. In F28 it should just sorta not do anything at all. It's not worth doing a big workaround to make F28 graphically add the repo in this case; let's just allow F28 to not actually add the repo. It's fine! This is fine. Signed-off-by: Adam Williamson --- tests/_check_install_source.pm | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/_check_install_source.pm b/tests/_check_install_source.pm index 0cb22f8a..360d9971 100644 --- a/tests/_check_install_source.pm +++ b/tests/_check_install_source.pm @@ -19,7 +19,9 @@ sub run { # check that the repo was used $self->root_console; - if ($addrepourl) { + # inst.addrepo doesn't actually work in < F29, so don't check it + my $version = lc(get_var("VERSION")); + if ($addrepourl && ($version eq 'rawhide' || $version > 28)) { if ($addrepourl =~ m,^nfs://,,) { # this line tells us it set up a repo for our URL... assert_script_run 'grep "repo addrepo.*' . ${addrepourl} . '" /tmp/packaging.log';