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

Fix an issue when both export-dir and tarball-dir is set #71

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mochrul
Copy link

@mochrul mochrul commented Aug 29, 2020

When the export-dir is set, the git tree is archived and then re-extracted
into the new place. Unfortunately, different directory is in use to create
the archive (output_dir) and then to extract it (tarball_dir, controlled by
the tarball-dir option). This means that if no orig.tar.gz is provided in
the tarball_dir directory, the extraction will fail.

@agx
Copy link
Owner

agx commented Aug 29, 2020

can you demonstrate this with a testcase tests/component/deb that fails initially and works after your fix?

@mochrul
Copy link
Author

mochrul commented Aug 29, 2020

WIP

When the export-dir is set, the git tree is archived and then re-extracted
into the new place.  Unfortunately, different directory is in use to create
the archive (output_dir) and then to extract it (tarball_dir, controlled by
the tarball-dir option).  This means that if no orig.tar.gz is provided in
the tarball_dir directory, the extraction will fail.
@agx
Copy link
Owner

agx commented Sep 9, 2020

I'm a bit puzzled since if you use overlay then isn't the whole point that we're not generating an archive but use tarball-dir?

@mochrul
Copy link
Author

mochrul commented Sep 10, 2020

Yes, you are completely right. To be honest, I inherited/stole the gbp configuration, and as it worked correctly for years, I haven't checked the actual meaning of the options. (It worked because the first step was to download the new upstream version and that step did generate the orig.tar.gz, so the issue is not triggered. This was the first time I started to work on the project in a different computer where I downloaded the upstream version).

To be honest, I tried to create the test case and reproduce the issue for a very long time before I realized that the overlay option also has its role in the game. And thinking the more about it the less I'm confident about the solution. But I'm still confident that this should be handled somehow, maybe with a correct error message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants