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 version bounds to all dependencies #119

Closed
ivanperez-keera opened this issue Jan 22, 2024 · 6 comments
Closed

Add version bounds to all dependencies #119

ivanperez-keera opened this issue Jan 22, 2024 · 6 comments
Assignees
Labels
CR:Status:Closed Admin only: Change request that has been completed CR:Type:Bug Admin only: Change request pertaining to error detected
Milestone

Comments

@ivanperez-keera
Copy link
Member

ivanperez-keera commented Jan 22, 2024

Description

The cabal packages do not currently specify version bounds. This makes hackage complain when we upload packages, and can create stability / installation issues for users.

Type

  • Bug: Cabal files are missing useful information.

Additional context

None.

Requester

  • Ivan Perez.

Method to check presence of bug

It's not possible to check automatically. The check is done by opening the cabal files and checking whether all dependencies of all components specify version bounds.

Expected result

All dependencies of all components specify version bounds.

Desired result

All dependencies of all components specify version bounds.

Proposed solution

Add version bounds to all dependencies of all components of all packages.

Further notes

None.

@ivanperez-keera ivanperez-keera changed the title Add version bounds to all dependencies. Add version bounds to all dependencies Jan 22, 2024
@ivanperez-keera ivanperez-keera added CR:Status:Initiated Admin only: Change request that has been initiated CR:Type:Bug Admin only: Change request pertaining to error detected labels Nov 23, 2024
@ivanperez-keera
Copy link
Member Author

Change Manager: Confirmed that the issue exists.

@ivanperez-keera ivanperez-keera added CR:Status:Confirmed Admin only: Change request that has been acknowledged by the change manager and removed CR:Status:Initiated Admin only: Change request that has been initiated labels Nov 23, 2024
@ivanperez-keera
Copy link
Member Author

Technical Lead: Confirmed that the issue should be addressed.

@ivanperez-keera ivanperez-keera added CR:Status:Accepted Admin only: Change request accepted by technical lead and removed CR:Status:Confirmed Admin only: Change request that has been acknowledged by the change manager labels Nov 23, 2024
@ivanperez-keera
Copy link
Member Author

Technical Lead: Issue scheduled for fixing in Ogma 1.6.0.

Fix assigned to: @ivanperez-keera .

@ivanperez-keera ivanperez-keera added CR:Status:Scheduled Admin only: Change requested scheduled and removed CR:Status:Accepted Admin only: Change request accepted by technical lead labels Nov 25, 2024
@ivanperez-keera ivanperez-keera added this to the 1.6.0 milestone Nov 25, 2024
@ivanperez-keera ivanperez-keera self-assigned this Nov 25, 2024
@ivanperez-keera ivanperez-keera added CR:Status:Implementation Admin only: Change request that is currently being implemented and removed CR:Status:Scheduled Admin only: Change requested scheduled labels Dec 24, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 24, 2024
The cabal packages do not currently specify version bounds. This makes hackage
complain when we upload packages, and can create stability / installation
issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 24, 2024
The cabal packages do not currently specify version bounds. This makes hackage
complain when we upload packages, and can create stability / installation
issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 24, 2024
…asa#119.

The cabal packages do not currently specify version bounds. This makes hackage
complain when we upload packages, and can create stability / installation
issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 24, 2024
…asa#119.

The cabal packages do not currently specify version bounds. This makes hackage
complain when we upload packages, and can create stability / installation
issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 24, 2024
.

The cabal packages do not currently specify version bounds. This makes hackage
complain when we upload packages, and can create stability / installation
issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 24, 2024
The cabal packages do not currently specify version bounds. This makes hackage
complain when we upload packages, and can create stability / installation
issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 24, 2024
The cabal packages do not currently specify version bounds. This makes hackage
complain when we upload packages, and can create stability / installation
issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 24, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 24, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 24, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 24, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 24, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
The cabal packages do not currently specify all version bounds. This
makes hackage complain when we upload packages, and can create stability
/ installation issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
The cabal packages do not currently specify all version bounds. This
makes hackage complain when we upload packages, and can create stability
/ installation issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
…asa#119.

The cabal packages do not currently specify all version bounds. This
makes hackage complain when we upload packages, and can create stability
/ installation issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
…asa#119.

The cabal packages do not currently specify all version bounds. This
makes hackage complain when we upload packages, and can create stability
/ installation issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
.

The cabal packages do not currently specify all version bounds. This
makes hackage complain when we upload packages, and can create stability
/ installation issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
The cabal packages do not currently specify all version bounds. This
makes hackage complain when we upload packages, and can create stability
/ installation issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
The cabal packages do not currently specify all version bounds. This
makes hackage complain when we upload packages, and can create stability
/ installation issues for users.

This commit adds lower and upper version bounds for all dependencies.
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
ivanperez-keera added a commit to ivanperez-keera/ogma that referenced this issue Dec 26, 2024
@ivanperez-keera
Copy link
Member Author

Implementor: Solution implemented, review requested.

@ivanperez-keera ivanperez-keera added CR:Status:Verification Admin only: Change request that is currently being verified and removed CR:Status:Implementation Admin only: Change request that is currently being implemented labels Dec 26, 2024
@ivanperez-keera
Copy link
Member Author

Change Manager: Verified that:

  • Solution is implemented:
     --- Dockerfile-ghc-8.4
     FROM ubuntu:trusty
     
     ENV DEBIAN_FRONTEND=noninteractive
     
     RUN apt-get update
     
     RUN apt-get install --yes software-properties-common
     RUN add-apt-repository ppa:hvr/ghc
     RUN apt-get update
     
     RUN apt-get install --yes ghc-8.4.4 cabal-install-2.4
     RUN apt-get install --yes libz-dev
     
     ENV PATH=/opt/ghc/8.4.4/bin:/opt/cabal/2.4/bin:$PWD/.cabal-sandbox/bin:$PATH
     
     RUN cabal update
     RUN cabal v1-sandbox init
     RUN cabal v1-install alex happy
     RUN apt-get install --yes git
     
     CMD git clone $REPO \
         && cd $NAME \
         && git checkout $COMMIT \
         && cd .. \
         && cabal v1-sandbox init \
         && cabal v1-install $NAME/$PAT**/ --constraint="aeson>=2.0.3.0" --enable-tests \
         && echo "Success"
    
     --- Dockerfile-ghc-9.4
     FROM ubuntu:focal
     
     RUN apt-get update
     
     RUN apt-get install --yes libz-dev
     RUN apt-get install --yes git
     
     # RUN curl --proto '=https' --tlsv1.2 -sSf https://get-ghcup.haskell.org | sh
     RUN apt-get install --yes wget
     RUN mkdir -p $HOME/.ghcup/bin
     RUN wget https://downloads.haskell.org/~ghcup/0.1.17.7/x86_64-linux-ghcup-0.1.17.7 -O $HOME/.ghcup/bin/ghcup
     
     RUN chmod a+x $HOME/.ghcup/bin/ghcup
     ENV PATH=$PATH:/root/.ghcup/bin/
     ENV PATH=$PATH:/root/.cabal/bin/
     RUN apt-get install --yes curl
     RUN apt-get install --yes gcc g++ make libgmp3-dev
     RUN ghcup install ghc 9.4
     RUN ghcup install cabal 3.2
     RUN ghcup set ghc 9.4.8
     RUN cabal update
     
     SHELL ["/bin/bash", "-c"]
     CMD git clone $REPO \
         && cd $NAME \
         && git checkout $COMMIT \
         && export PATH=$PATH:$PWD/.cabal-sandbox/bin/ \
         && cabal v1-sandbox init \
         && cabal v1-install alex happy --constraint="happy<2.0" \
         && cabal v1-install BNFC ogma-**/ --constraint="aeson>=2.0.3.0" \
         && cabal v1-install ogma-**/ --constraint="aeson>=2.0.3.0" --enable-tests \
         && echo "Success"
    Command (substitute variables based on new path after merge):
    $ docker run -e "REPO=https://github.com/ivanperez-keera/ogma" -e "NAME=ogma" -e "PAT=ogma"  -e "COMMIT=cc38a221e10bf85edde7b54b24720cb048aa8db9" -it ogma-verify-119-ghc-8.4
    $ docker run -e "REPO=https://github.com/ivanperez-keera/ogma" -e "NAME=ogma" -e "PAT=ogma"  -e "COMMIT=cc38a221e10bf85edde7b54b24720cb048aa8db9" -it ogma-verify-119-ghc-9.4
    
  • Implementation is documented. Details:
    No updates needed.
  • Change history is clear.
  • Commit messages are clear.
  • Changelogs are updated.
  • Examples are updated. Details:
    No updates needed.
  • Required version bumps are evaluated. Details:
    No bump needed; although we are restricting versions of dependencies, the way they are set simply captures the current compatibility of Ogma with other libraries and does not restrict it further.

@ivanperez-keera
Copy link
Member Author

Change Manager: Implementation ready to be merged.

@ivanperez-keera ivanperez-keera added CR:Status:Closed Admin only: Change request that has been completed and removed CR:Status:Verification Admin only: Change request that is currently being verified labels Dec 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CR:Status:Closed Admin only: Change request that has been completed CR:Type:Bug Admin only: Change request pertaining to error detected
Projects
None yet
Development

No branches or pull requests

1 participant