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

pod trunk push fails with "Git sources should specify a tag", even with --allow-warnings #2537

Closed
michaeltyson opened this issue Sep 29, 2014 · 8 comments

Comments

@michaeltyson
Copy link

We have a Pod that comes from a dumb HTTP git repository, which does not support --depth.

From August, CocoaPods has added shallow fetching (--depth 1) to its git support, which breaks our Pod. As a workaround, I've replaced the ":tag" parameter with ":commit", to force CocoaPods to avoid the incompatible --depth parameter.

However, trying to push this fixed Pod results in the failure:

    - WARN  | [source] Git sources should specify a tag.
[!] The Pod Specification did not pass validation.

Trying to add the --allow-warnings param to pod trunk push makes no difference, even though the issue is a warning.

@segiddins
Copy link
Member

Currently, trunk does not accept specs with warnings. In order to debug the issue with the downloader, we need to be able to play around with such a 'dumb HTTP git repository'.

@michaeltyson
Copy link
Author

Certainly - the repository is http://developer.audiob.us/download/SDK.git

You'll need to make a (free) account at http://developer.audiob.us/register to get an email/password for access (this is a measure we take to ensure developers are using our system correctly).

Until then, do you have any thoughts on how I might proceed? Currently, the Pod is unusable.

@segiddins
Copy link
Member

This is an issue with the downloader, and thus needs to be rectified over there. Can you provide me with credentials to access the repo? (my email is on my profile)

@michaeltyson
Copy link
Author

Okay - I've registered you; you'll need to use the password reset facility to get a password (check your inbox).

Thank you!

@segiddins
Copy link
Member

@kylef
Copy link
Contributor

kylef commented Oct 7, 2014

We've made a fix for the dumb downloader.

Can you try updating the downloader and confirming the fix:

$ [sudo] gem install cocoapods-downloader

@ninjitaru
Copy link

Hi,

Is it possible to have option to manually remove --depth 1 using pod spec.
The repository that I have to work with is even more dumb.
It actually crash itself when --depth is used, which is a disaster
I tried query our repo's git Content-Type, which appear to be smart
Content-Type: application/x-git-upload-pack-advertisement
so there is something more wrong with our repo...

@segiddins
Copy link
Member

Jason, please open up a new issue with some more details.

-Samuel E. Giddins

On Dec 19, 2014, at 6:14 AM, Jason Chang notifications@github.com wrote:

Hi,

Is it possible to have option to manually remove --depth 1 using pod spec.
The repository that I have to work with is even more dumb.
It actually crash itself when --depth is used, which is a disaster


Reply to this email directly or view it on GitHub.

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

No branches or pull requests

4 participants