Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fails for packages with missing or empty prep section #46

Open
dhodovsk opened this issue Sep 3, 2020 · 2 comments
Open

Fails for packages with missing or empty prep section #46

dhodovsk opened this issue Sep 3, 2020 · 2 comments
Labels
kind/feature New feature or a request for enhancement.

Comments

@dhodovsk
Copy link
Contributor

dhodovsk commented Sep 3, 2020

Packages:

  • c8s: appstream-data, filesystem, qt5-doc
  • c8: perl-srpm-macros, qt5-doc, rootfiles, rteval-loads, spice-client-win, spice-qxl-xddm, spice-vdagent-win, centos-release, filesystem, appstream-data, go-compilers, google-roboto-slab-fonts

Convert log:

[dhodovsk@shiny playground]$ podman run --rm -v $PWD:/workdir:z dist2src dist2src -vv convert rpms/appstream-data:c8s src/appstream-data:c8s
convert(origin='rpms/appstream-data:c8s', dest='src/appstream-data:c8s')
Running command get_sources.sh in /workdir/rpms/appstream-data
output = Retrieving https://git.centos.org/sources/appstream-data/c8s/c01fab7926a0fd4f2b1196600b5dd1b2ad4b5425
Retrieving https://git.centos.org/sources/appstream-data/c8s/6b08557071ff73d81cc4a4bbab5908ef3e21238f
Retrieving https://git.centos.org/sources/appstream-data/c8s/d22ded3cd4471c1db4835f7cdf01a4b8317f69f1
Retrieving https://git.centos.org/sources/appstream-data/c8s/811c4fe5e36dde125dfb0d3a48a6fc9dce80192c

Running rpmbuild in /workdir/rpms/appstream-data
Traceback (most recent call last):
  File "/usr/local/bin/dist2src", line 11, in <module>
    load_entry_point('dist-git-to-source-git==0.1.dev123+gc102801.d20200903', 'console_scripts', 'dist2src')()
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/dist2src/cli.py", line 82, in wrapper
    ret = func(*args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/dist2src/cli.py", line 208, in convert
    d2s.convert(origin_branch, dest_branch)
  File "/usr/local/lib/python3.6/site-packages/dist2src/core.py", line 302, in convert
    self.run_prep()
  File "/usr/local/lib/python3.6/site-packages/dist2src/core.py", line 258, in run_prep
    self._enforce_autosetup()
  File "/usr/local/lib/python3.6/site-packages/dist2src/core.py", line 220, in _enforce_autosetup
    for i, line in enumerate(prep_lines):
TypeError: 'NoneType' object is not iterable
@TomasTomecek
Copy link
Member

Yeah, just ran into this - funny.

Well, since we rely on %[auto]setup to unpack the tarball and create a git repo we'd need to do those ourselves.

@TomasTomecek TomasTomecek added the kind/feature New feature or a request for enhancement. label Sep 4, 2020
TomasTomecek added a commit to TomasTomecek/dist-git-to-source-git that referenced this issue Sep 4, 2020
Related packit#46

Signed-off-by: Tomas Tomecek <ttomecek@redhat.com>
csomh pushed a commit to TomasTomecek/dist-git-to-source-git that referenced this issue Sep 9, 2020
Related packit#46

Signed-off-by: Tomas Tomecek <ttomecek@redhat.com>
@dhodovsk dhodovsk changed the title Fails for package with no prep section Fails for package with missing or empty prep section Sep 22, 2020
@dhodovsk dhodovsk changed the title Fails for package with missing or empty prep section Fails for packages with missing or empty prep section Sep 22, 2020
@sentry-io
Copy link

sentry-io bot commented Feb 8, 2021

Sentry issue: DIST2SRC-UPDATE-1Q

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature or a request for enhancement.
Projects
None yet
Development

No branches or pull requests

2 participants