Skip to content

Commit 3f5b304

Browse files
committed
Link all packages to index.ros.org
Before it was going to declared urls if available. But most didn't have urls declared. And for this purpose the index with the dependencies is the most valuable.
1 parent 9cc156f commit 3f5b304

File tree

2 files changed

+4
-7
lines changed

2 files changed

+4
-7
lines changed

ros_buildfarm/status_page.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,7 @@ def build_release_status_page(
124124
template_name = 'status/release_status_page.html.em'
125125
data = {
126126
'title': 'ROS packages for %s' % rosdistro_name.capitalize(),
127+
'rosdistro_name': rosdistro_name,
127128
'start_time': start_time,
128129
'start_time_local_str': time.strftime('%Y-%m-%d %H:%M:%S %z', time.localtime(start_time)),
129130

@@ -235,6 +236,7 @@ def build_repos_status_page(
235236
data = {
236237
'title': 'All packages for %s targets' % rosdistro_name.capitalize(),
237238
'start_time': start_time,
239+
'rosdistro_name': rosdistro_name,
238240
'start_time_local_str': time.strftime('%Y-%m-%d %H:%M:%S %z', time.localtime(start_time)),
239241

240242
'resource_hashes': get_resource_hashes(),
@@ -1075,6 +1077,7 @@ def build_release_compare_page(
10751077
data = {
10761078
'title': 'ROS packages in %s' % ' '.join([x.capitalize() for x in rosdistro_names]),
10771079

1080+
'rosdistro_name': rosdistro_name,
10781081
'start_time': start_time,
10791082
'start_time_local_str': time.strftime('%Y-%m-%d %H:%M:%S %z', time.localtime(start_time)),
10801083

ros_buildfarm/templates/status/release_status_page.html.em

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -120,13 +120,7 @@
120120
@
121121
<td>@
122122
@ <div>@
123-
@ @ @[if pkg.url]@
124-
@ @ <a href="@pkg.url">@
125-
@ @ @[end if]@
126-
@ @ @pkg.name@
127-
@ @ @[if pkg.url]@
128-
@ @ </a>@
129-
@ @ @[end if]@
123+
@ @ <a href="https://index.ros.org/p/@pkg.url#@rosdistro_name">@pkg.name</a>
130124
@ </div>@
131125
@{
132126
hidden_texts = []

0 commit comments

Comments
 (0)