Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

add jpeg to OpenBSD prereq list #4193

Merged
merged 2 commits into from Nov 17, 2018
Merged

add jpeg to OpenBSD prereq list #4193

merged 2 commits into from Nov 17, 2018

Conversation

kivikakk
Copy link
Contributor

Without the jpeg package installed, this occurs while installing the matrix-synapse Python package:

    The headers or library files could not be found for jpeg,
    a required dependency when compiling Pillow from source.

    Please see the install instructions at:
       https://pillow.readthedocs.io/en/latest/installation.html

    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-eSNi9N/pillow/setup.py", line 814, in <module>
        raise RequiredDependencyException(msg)
    __main__.RequiredDependencyException:

    The headers or library files could not be found for jpeg,
    a required dependency when compiling Pillow from source.

    Please see the install instructions at:
       https://pillow.readthedocs.io/en/latest/installation.html

@kivikakk kivikakk changed the base branch from master to develop November 16, 2018 04:35
@kivikakk
Copy link
Contributor Author

kivikakk commented Nov 16, 2018

Guess who didn't read the contributing guidelines first? 🤦‍♀️ Retargetted develop, added sign-off.

@hawkowl
Copy link
Contributor

hawkowl commented Nov 16, 2018

@kivikakk Hi! One other thing you need is a changelog entry -- CONTRIBUTING has more details, but you want to make a file called <PR number>.bugfix containing details, so people know it works now.

Thanks!

Signed-off-by: Ashe Connor <ashe@kivikakk.ee>
@kivikakk
Copy link
Contributor Author

@hawkowl Thanks for your patience! (And hello 😊) I've added the entry.

@hawkowl hawkowl merged commit 4285c81 into matrix-org:develop Nov 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants