artifactory-debian icon indicating copy to clipboard operation
artifactory-debian copied to clipboard

plugin fails with dput v0.12.1 on Debian Stretch

Open bollenn opened this issue 6 years ago • 4 comments

This plugin fails with dput v0.12.1.

Seems the install path now shall be /usr/share/dput/dput/methods/webdav.py but besides changing this path and the helper import some more errors are popping up...

bollenn avatar Mar 12 '18 12:03 bollenn

Please provide platform information (lsb_release -a) and/or "apt-cache policy dput".

jhermann avatar Mar 12 '18 16:03 jhermann

I can reproduce:

$ apt-cache policy dput
dput:
  Installed: 0.12.1
  Candidate: 0.12.1
  Version table:
 *** 0.12.1 500
        500 http://debian.lth.se/debian stretch/main amd64 Packages
        500 http://debian.lth.se/debian stretch/main i386 Packages
        100 /var/lib/dpkg/status

Just like @bollenn, I worked-around some issues, but the error which I have had problems solving is that the dput.config (aka __main__.config) is no longer available:

Traceback (most recent call last):
  File "/usr/bin/dput", line 11, in <module>
    load_entry_point('dput==0.12.1', 'console_scripts', 'execute-dput')()
  File "/usr/share/dput/dput/dput.py", line 1153, in main
    files_to_upload, debug, 0, progress=progress)
  File "/usr/share/dput/dput/methods/webdav.py", line 365, in upload
    host_config, cli_params = _get_config_data(fqdn)
  File "/usr/share/dput/dput/methods/webdav.py", line 349, in _get_config_data
    host_argument = _get_host_argument(fqdn)
  File "/usr/share/dput/dput/methods/webdav.py", line 317, in _get_host_argument
    config = dput.config # pylint: disable=no-member
AttributeError: 'module' object has no attribute 'config'

The config object only lives in a local scope. Only by removing the use of _get_config_data() from upload() I was able to get it working. But I don't understand what I broke by doing so :-).

olof avatar Apr 13 '18 14:04 olof

@olof where you able to get all issues fixed?

bollenn avatar Feb 19 '19 19:02 bollenn

@bollenn Sorry for late reply; yes, with the caveat of my last comment: i just ripped of something that i didn't understand the need for, with the confidence of somebody who thinks they know what they are doing, but clearly don't :). But it worked for my use case/in my environment. No longer a user of Artifactory today.

olof avatar Nov 28 '19 21:11 olof