Skip to content
Snippets Groups Projects

Reduce number of fog gem dependencies

Merged Stan Hu requested to merge reduce-fog-deps into master

Currently supported:

  • AWS
  • Azure
  • OpenStack
  • Google
  • local storage

Closes #15352 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Stan Hu Milestone changed to %8.9

    Milestone changed to %8.9

  • Author Owner

    @connorshea Had trouble including fog-azure-rm due to a dependency issue with mime-types:

    undler could not find compatible versions for gem "mime-types":
      In snapshot (Gemfile.lock):
        mime-types (= 2.99.1)
    
      In Gemfile:
        carrierwave (~> 0.10.0) was resolved to 0.10.0, which depends on
          mime-types (>= 1.16)
    
        carrierwave (~> 0.10.0) was resolved to 0.10.0, which depends on
          mime-types (>= 1.16)
    
        github-linguist (~> 4.7.0) was resolved to 4.7.6, which depends on
          mime-types (>= 1.19)
    
        gollum-lib (~> 4.1.0) was resolved to 4.1.0, which depends on
          gollum-grit_adapter (~> 1.0) was resolved to 1.0.0, which depends on
            gitlab-grit (>= 2.7.1, ~> 2.7) was resolved to 2.8.1, which depends on
              mime-types (< 3, >= 1.16)
    
        gollum-rugged_adapter (~> 0.4.2) was resolved to 0.4.2, which depends on
          mime-types (>= 1.15)
    
        rails (= 4.2.6) was resolved to 4.2.6, which depends on
          actionmailer (= 4.2.6) was resolved to 4.2.6, which depends on
            mail (>= 2.5.4, ~> 2.5) was resolved to 2.6.4, which depends on
              mime-types (< 4, >= 1.16)
    
        fog-azure-rm (~> 0.0) was resolved to 0.0.0, which depends on
          rest-client (~> 1.6.9) was resolved to 1.6.9, which depends on
            mime-types (~> 1.16)
    
        tinder (~> 1.10.0) was resolved to 1.10.1, which depends on
          mime-types
  • Stan Hu Changed title: WIP: Reduce number of gem dependencies by only usng the fog gems we needWIP: Reduce number of fog gem dependencies

    Changed title: WIP: Reduce number of gem dependencies by only usng the fog gems we needWIP: Reduce number of fog gem dependencies

  • Stan Hu Added 1 commit:

    Added 1 commit:

    • d315ef7e - Reduce number of fog gem dependencies
  • Stan Hu mentioned in issue #15352 (closed)

    mentioned in issue #15352 (closed)

  • @stanhu have we tried talking to the maintainers of the fog-azure gem about this? Presumably it's the same people as fog-core, and they were responsive when I've talked to them.

    Edited by Connor Shea
  • Author Owner

    @connorshea I think we just need to have them loosen their requirements to allow mime-types 2.x. Can I assign this issue to you?

  • Reassigned to @connorshea

  • @stanhu I'll see what I can do.

  • Reassigned to @stanhu

  • We decided on going with fog-azure, since that does the same thing without dependency conflicts.

  • Stan Hu Added 55 commits:

    Added 55 commits:

    • d315ef7e...2d084dd8 - 54 commits from branch master
    • cafcc149 - Reduce number of fog gem dependencies
  • You should probably include fog-core explicitly so we can handle upgrading it.

  • Stan Hu Added 1 commit:

    Added 1 commit:

    • c95c0edb - Reduce number of fog gem dependencies
  • Stan Hu Unmarked this merge request as a Work In Progress

    Unmarked this merge request as a Work In Progress

  • Stan Hu Added 13 commits:

    Added 13 commits:

    • c95c0edb...e9bb895b - 12 commits from branch master
    • e5bb417c - Reduce number of fog gem dependencies
  • Connor Shea
    Connor Shea @connorshea started a thread on commit e5bb417c
214 211 flowdock (0.7.1)
215 212 httparty (~> 0.7)
216 213 multi_json
217 fog (1.36.0)
  • Reassigned to @rspeicher

  • Yusssssss. :heart:

  • Robert Speicher Status changed to merged

    Status changed to merged

  • Robert Speicher mentioned in commit e6389dfe

    mentioned in commit e6389dfe

  • Stan Hu mentioned in merge request !4444 (merged)

    mentioned in merge request !4444 (merged)

  • mentioned in issue #14043 (moved)

  • Rémy Coutable mentioned in commit 722211cd

    mentioned in commit 722211cd

  • Benjamin Albrecht mentioned in merge request !4928 (closed)

    mentioned in merge request !4928 (closed)

  • Tim Tilberg mentioned in merge request !5783 (closed)

    mentioned in merge request !5783 (closed)

  • Rémy Coutable Mentioned in merge request !4928 (closed)

    Mentioned in merge request !4928 (closed)

  • Please register or sign in to reply
    Loading