Add comment

mamba-org · Dec 26, 2024 · f69c65c · f69c65c
1 parent fa7ee00
commit f69c65c
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/libmamba/src/core/package_fetcher.cpp b/libmamba/src/core/package_fetcher.cpp
@@ -325,6 +325,9 @@ namespace mamba
 
     std::string PackageFetcher::channel() const
     {
+        // Note that in the oci case, the pkg url also implicitly starts with "https://"
+        // but should be included in the first case (with the url containing "ghcr" condition)
+        // TODO We should maybe think of a cleaner way of differentiating all this
         if (!util::starts_with(m_package_info.package_url, "file://")
             && (!util::starts_with(m_package_info.package_url, "https://")
                 || util::contains(m_package_info.package_url, "ghcr")))
@@ -341,6 +344,9 @@ namespace mamba
 
     std::string PackageFetcher::url_path() const
     {
+        // Note that in the oci case, the pkg url also implicitly starts with "https://"
+        // but should be included in the first case (with the url containing "ghcr" condition)
+        // TODO We should maybe think of a cleaner way of differentiating all this
         if (!util::starts_with(m_package_info.package_url, "file://")
             && (!util::starts_with(m_package_info.package_url, "https://")
                 || util::contains(m_package_info.package_url, "ghcr")))