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

Consider multiple distribution files with REP-143 #330

Merged
merged 1 commit into from
Dec 4, 2014
Merged

Conversation

wjwwood
Copy link
Contributor

@wjwwood wjwwood commented Dec 1, 2014

With REP-143 (ros-infrastructure/rep#87) there can be multiple distribution files, so bloom with try to use the new api in rosdistro (ros-infrastructure/rosdistro#45) to get a list of the distribution files and then take the last one. Otherwise it will fall back to the current rosdistro API, so this change is backwards compatible.

@wjwwood
Copy link
Contributor Author

wjwwood commented Dec 2, 2014

This works against the current rosdistro (repo and library).

@dirk-thomas for review.

@dirk-thomas
Copy link
Member

lgtm

@tfoote
Copy link
Member

tfoote commented Dec 2, 2014

+1, though once we're finished the transition we should use a versioned dependency and remove the backwards compatibility.

@dirk-thomas
Copy link
Member

There is no strong reason to remove the REP 141 support anytime soon. rosdistro if supporting both formats too.

@tfoote
Copy link
Member

tfoote commented Dec 2, 2014

I'm not talking about removing REP 141 support. I'm talking about removing the deprecated API support with the try catch logic. Once there's an released version of rosdistro which supports 141 and 143 there's no need for the try catch logic.

@dirk-thomas
Copy link
Member

Yes, as soon as rosdistro is released the try/except can be removed in favor of a versioned dependency.

wjwwood added a commit that referenced this pull request Dec 4, 2014
Consider multiple distribution files with REP-143
@wjwwood wjwwood merged commit 8f9e531 into master Dec 4, 2014
@wjwwood wjwwood deleted the rep_143 branch December 4, 2014 17:40
@wjwwood
Copy link
Contributor Author

wjwwood commented Dec 4, 2014

I created a ticket to track this: #334

@tfoote
Copy link
Member

tfoote commented Dec 6, 2014

I've tested up to the pull-request stage and get this:

Traceback (most recent call last):
  File "/tmp/deploy_ros_buildfarm/venv/local/lib/python2.7/site-packages/bloom/commands/release.py", line 1058, in perform_release
    pull_request_url = open_pull_request(track, repository, distro, interactive)
  File "/tmp/deploy_ros_buildfarm/venv/local/lib/python2.7/site-packages/bloom/commands/release.py", line 665, in open_pull_request
    updated_distribution_file = generate_ros_distro_diff(track, repository, distro)
  File "/tmp/deploy_ros_buildfarm/venv/local/lib/python2.7/site-packages/bloom/commands/release.py", line 473, in generate_ros_distro_diff
    distro_file_name = get_relative_distribution_file_path(distro)
  File "/tmp/deploy_ros_buildfarm/venv/local/lib/python2.7/site-packages/bloom/commands/release.py", line 344, in get_relative_distribution_file_path
    distribution_file_url = urlparse(get_disitrbution_file_url(distro))
  File "/usr/lib/python2.7/urlparse.py", line 143, in urlparse
    tuple = urlsplit(url, scheme, allow_fragments)
  File "/usr/lib/python2.7/urlparse.py", line 176, in urlsplit
    cached = _parse_cache.get(key, None)
TypeError: unhashable type: 'list'

Failed to open pull request: TypeError - unhashable type: 'list'

@wjwwood wjwwood restored the rep_143 branch February 4, 2015 00:34
@clalancette clalancette deleted the rep_143 branch August 27, 2019 18:36
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.

3 participants