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

Add OpenEmbedded to system deps, limit display of non-tiered platforms #440

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

rkent
Copy link

@rkent rkent commented Oct 26, 2024

This PR is an intermediate step in a longer-term plan (see #415) to remove the paged listing pages for packages and dependencies, and the separate search pages, and replace them with pages that are implemented as a scrolled, searchable lists using Tabulator. That is implemented for packages in PR #438.

I want to eliminate the old combined packages & dependencies search pages from rosindex, but I cannot do that until I implement system dependency search in Tabulator. This PR is an intermediate simplification prior to doing that.

In #415, I mention that the system dependency list is one of three core functionalities of rosindex, along with the particulars of items in the lists. This list is fairly buried in current rosindex (under INDEX/system dependencies in the header), but has a much more prominent position in my proposed format, which you can see at devindex

The current list (which was mostly broken until recently, see #416), displays a long list of platform types with no formal tier support within ROS. This PR:

  • Only displays current platforms (with a listed Tier in REP2000) on the System Dependencies list page. (This includes Debian, MacOS, OpenEmbedded [which is newly displayed], RHEL, and Ubuntu; excluded now is arch, centos, cygwin, fedora, freebsd, mandrake, opensuse). Note the excluded platforms are still visible on the system dependency detail page. Windows is not shown as their package dependency management is complex, and not currently supported in rosindex).
  • The displayed name on the system dependency list page is now set separately from the platform identifier. Thus osx becomes MacOS for example. This does not affect search at the moment, but will in a future update (along with the currently not-searched, and relatively recently added, description).

The result of this can be seen (within a few hours of this PR request appearing) at https://index.rosdabbler.com/deps/ and can be seen with a few additional commits that have not currently released as PRs (such as newly-included PIP descriptions) at https://devindex.rosdabbler.com/deps/

@rkent
Copy link
Author

rkent commented Oct 26, 2024

I noticed something strange, MacOS is checked as available everywhere. I'll need to investigate, so please hold any review of this.

@rkent rkent force-pushed the openembedded-in-deps branch from 12af299 to f6278ba Compare October 27, 2024 15:15
@rkent
Copy link
Author

rkent commented Oct 27, 2024

OK this is fixed now.

@rkent rkent force-pushed the openembedded-in-deps branch from f6278ba to 554bd6b Compare October 31, 2024 16:37
Copy link
Member

@tfoote tfoote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks focusing this display makes sense.

@tfoote tfoote merged commit fd5205c into ros-infrastructure:ros2 Nov 8, 2024
1 check passed
@rkent rkent deleted the openembedded-in-deps branch November 20, 2024 03:34
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