Bug 1761626 - squashfuse-devel is missing header files
Summary: squashfuse-devel is missing header files
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: squashfuse
Version: 32
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Kyle Fazzari
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: SIMPLE_PATCHES
TreeView+ depends on / blocked
 
Reported: 2019-10-14 23:15 UTC by "FeRD" (Frank Dana)
Modified: 2021-01-18 02:20 UTC (History)
2 users (show)

Fixed In Version: squashfuse-0.1.102-7.fc33 squashfuse-0.1.102-7.fc32
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-01-18 01:33:52 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github vasi squashfuse pull 36 0 None closed Move headers to subdir of include/, install missing headers 2021-01-09 14:31:03 UTC

Description "FeRD" (Frank Dana) 2019-10-14 23:15:49 UTC
Description of problem:
/usr/include/squashfuse.h contains the following #includes:

#include "dir.h"
#include "file.h"
#include "fs.h"
#include "traverse.h"
#include "util.h"
#include "xattr.h"

(In fact, that's _all_ it contains.)

However, none of those files are packaged, making the header unusable.


Version-Release number of selected component (if applicable):
squashfuse-devel-0.1.102-3.fc30.x86_64


How reproducible:
100%


Steps to Reproduce:
1. Attempt to #include "squashfuse.h"
2.
3.


Actual results:

In file included from /home/ferd/rpmbuild/REPOS/AppImageKit/lib/libappimage/src/libappimage/libappimage.c:44:
/usr/include/squashfuse.h:28:10: fatal error: dir.h: No such file or directory
   28 | #include "dir.h"
      |          ^~~~~~~
compilation terminated.
make[2]: *** [src/libappimage/CMakeFiles/libappimage.dir/build.make:63: src/libappimage/CMakeFiles/libappimage.dir/libappimage.c.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:305: src/libappimage/CMakeFiles/libappimage.dir/all] Error 2
make: *** [Makefile:130: all] Error 2


Expected results:
No compilation errors due to missing includes.


Additional info:

The referenced include files _are_ present in the squashfuse source distribution, see https://github.com/vasi/squashfuse

However, it looks like upstream isn't properly installing its includes, since:

$ pkg-config squashfuse --variable=includedir
/usr/include

...I would think that would be /usr/include/squashfuse/ (which does not currently exist), a path which would then contain **all** of the necessary header files.

Comment 1 "FeRD" (Frank Dana) 2019-10-17 05:52:21 UTC
Note that I have submitted a PR upstream to correct the header installation:

https://github.com/vasi/squashfuse/pull/36

Comment 2 Kyle Fazzari 2019-10-31 15:43:39 UTC
Thank you for the report, and I appreciate the effort to fix this upstream first. In your opinion, should we wait for that fix to land, or patch it while we wait?

Comment 3 "FeRD" (Frank Dana) 2019-10-31 17:13:53 UTC
There hasn't been a commit upstream since June 2018, and no reaction to my PR in a full two weeks. It's starting to seem like squashfuse is effectively an abandoned project, I doubt waiting on action there would prove fruitful.

Comment 4 "FeRD" (Frank Dana) 2019-11-05 14:31:14 UTC
Well, turns out I was wrong. As of 4 hours ago, my PR is merged upstream. So updating from the git HEAD at https://github.com/vasi/squashfuse/ is another option.

Comment 5 Ben Cotton 2020-02-11 17:49:12 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 32 development cycle.
Changing version to 32.

Comment 6 Artur Frenszek-Iwicki 2020-06-04 12:38:53 UTC
So... any plans on moving this forward? I've just been bitten by this myself.

Comment 7 Kyle Fazzari 2020-06-04 16:16:44 UTC
I was hoping upstream would cut a new release containing the fix, but it's admittedly been a while.

Comment 8 Artur Frenszek-Iwicki 2020-06-04 20:19:58 UTC
Personally I'm not pushing to adopt the latest git snapshot; cherry-picking only the patch for missing headers would be fine.

Comment 9 "FeRD" (Frank Dana) 2020-10-19 10:18:00 UTC
Artur's src.fpo PR, which addresses this by cherry-picking the commit from my upstream PR as a patchfile applied in squashfuse.spec, is here:

https://src.fedoraproject.org/rpms/squashfuse/pull-request/2

(Please do consider merging the PR and releasing an updated squashfuse RPM. I think waiting on action from upstream is an nice idea whose time has long since passed.)

Comment 10 Artur Frenszek-Iwicki 2020-11-01 11:49:07 UTC
Kyle, it's been over a year since this issue was originally reported. While the squashfuse package isn't of much importance in Fedora, having squashfuse-devel in this broken state prevents anyone from packaging any new software that would depend on the library.

Please review the pull requests and merge one of them, or maybe write your own patch. I am setting the NEEDINFO flag. If you do not respond within a week, I will request help from a Proven Packager via the Simple Patch Policy (https://fedoraproject.org/wiki/Policy_for_simple_patches).

Comment 11 Kyle Fazzari 2020-12-16 03:04:35 UTC
I'm afraid I'm unable to do this at this time. Please do use the simple patch policy.

Comment 12 "FeRD" (Frank Dana) 2020-12-16 03:56:16 UTC
@suve
I took the liberty of performing the required scratch builds:

ScratchBuild[rawhide]=https://koji.fedoraproject.org/koji/taskinfo?taskID=57542036
ScratchBuild[f33]=https://koji.fedoraproject.org/koji/taskinfo?taskID=57542045
ScratchBuild[f32]=https://koji.fedoraproject.org/koji/taskinfo?taskID=57542063

Comment 14 Fedora Update System 2021-01-09 23:30:42 UTC
FEDORA-2021-eb68661994 has been submitted as an update to Fedora 33. https://bodhi.fedoraproject.org/updates/FEDORA-2021-eb68661994

Comment 15 Fedora Update System 2021-01-09 23:30:42 UTC
FEDORA-2021-b07c548687 has been submitted as an update to Fedora 32. https://bodhi.fedoraproject.org/updates/FEDORA-2021-b07c548687

Comment 16 Fedora Update System 2021-01-10 01:35:16 UTC
FEDORA-2021-eb68661994 has been pushed to the Fedora 33 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2021-eb68661994`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-eb68661994

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 17 Fedora Update System 2021-01-10 02:17:49 UTC
FEDORA-2021-b07c548687 has been pushed to the Fedora 32 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2021-b07c548687`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-b07c548687

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 18 Fedora Update System 2021-01-18 01:33:52 UTC
FEDORA-2021-eb68661994 has been pushed to the Fedora 33 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 19 Fedora Update System 2021-01-18 02:20:23 UTC
FEDORA-2021-b07c548687 has been pushed to the Fedora 32 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.