#148 _print_old_checkout_warning should be more advertised
Closed: Fixed 7 years ago Opened 7 years ago by sergiomb.

Hi,
This warning (1) is very useful is the fix of git repos without namespace but only appears on fedpkg build command, not in fedpkg push neither in switch-branch, when we deal with git, git pull etc we get an warning from git server: WARNING: 'foo' is an alias for 'rpms/foo' ) , when we have _print_old_checkout_warning function which prints the way to fix the warning .
maybe one new option rpkg check ? is stupid do fedpkg build to get the message :) .
Cheers.
(1)

  def _print_old_checkout_warning(self, module):
       self.log.warning('Your git configuration does not use a namespace.')
       self.log.warning('Consider updating your git configuration by running:')
       self.log.warning('  git remote set-url %s %s',
                     self.branch_remote, self._get_namespace_giturl(module))

@sergiomb in which repo you get such warning WARNING: 'foo' is an alias for 'rpms/foo'?

every old clones done by fedpkg or for example :

git clone ssh://sergiomb@pkgs.fedoraproject.org/azureus

this just happens with old clones that I have in my disk I didn't start develop yesterday

I'm going to add the warning to each operation with dist-git, pull and push. I'm don't quite understand why that warning is given when build, as Koji may not care about the namespace in dist-git I think. What do you think? @sergiomb

Hi, Looks good to me , IMHO the more you add the better, for example add also to switch-branch, srpm .

Thanks.

@cqi changed the status to Closed

7 years ago

Login to comment on this ticket.

Metadata