rpkg

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

Closed: Fixed
Reopen Issue

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.

Fixed in PR #159

@cqi changed the status to Closed
Metadata
None
None
None
None
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.