After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 721167 - install-check is silly with DESTDIR installs
install-check is silly with DESTDIR installs
Status: RESOLVED FIXED
Product: jhbuild
Classification: Infrastructure
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: Jhbuild maintainers
Jhbuild QA
: 665085 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2013-12-28 20:06 UTC by Allison Karlitskaya (desrt)
Modified: 2015-11-10 09:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
install-check: Drop as no longer useful and potentially dangerous (8.71 KB, patch)
2015-11-10 00:15 UTC, Emanuele Aina
committed Details | Review
install-check: Drop as no longer useful and potentially dangerous (8.71 KB, patch)
2015-11-10 09:58 UTC, Emanuele Aina
committed Details | Review

Description Allison Karlitskaya (desrt) 2013-12-28 20:06:15 UTC
We should drop install-check.  It's totally pointless when we're installing to a DESTDIR and copying the files into place.

It's also potentially problematic when mixed with .pyc files in the case that we don't do a DESTDIR install (for some reason).  The .pyc file can be updated (due to the fact that it contains a timestamp) and will therefore be installed, but the original .py file wouldn't be because it's the same.  This means that the timestamp in the .pyc file would mismatch the installed .py file (which would either be equal to the timestamp that it was checked out of git, or that it was generated, if that is the case).
Comment 1 Allison Karlitskaya (desrt) 2014-01-02 05:52:05 UTC
*** Bug 665085 has been marked as a duplicate of this bug. ***
Comment 2 Emanuele Aina 2015-11-10 00:15:55 UTC
Created attachment 315163 [details] [review]
install-check: Drop as no longer useful and potentially dangerous

`install-check` is totally pointless when installing to a DESTDIR and
copying the files into place, which nowadays is JHBuild's normal method
of operation.

It's also potentially problematic when mixed with .pyc files in the case
that we don't do a DESTDIR install (for some reason).  The .pyc file can
be updated (due to the fact that it contains a timestamp) and will
therefore be installed, but the original .py file wouldn't be because
it's the same.  This means that the timestamp in the .pyc file would
mismatch the installed .py file (which would either be equal to the
timestamp that it was checked out of git, or that it was generated, if
that is the case).
Comment 3 Colin Walters 2015-11-10 02:10:11 UTC
Review of attachment 315163 [details] [review]:

Right.
Comment 4 Emanuele Aina 2015-11-10 09:58:47 UTC
The following fix has been pushed:
18aaf92 install-check: Drop as no longer useful and potentially dangerous
Comment 5 Emanuele Aina 2015-11-10 09:58:56 UTC
Created attachment 315179 [details] [review]
install-check: Drop as no longer useful and potentially dangerous

`install-check` is totally pointless when installing to a DESTDIR and
copying the files into place, which nowadays is JHBuild's normal method
of operation.

It's also potentially problematic when mixed with .pyc files in the case
that we don't do a DESTDIR install (for some reason).  The .pyc file can
be updated (due to the fact that it contains a timestamp) and will
therefore be installed, but the original .py file wouldn't be because
it's the same.  This means that the timestamp in the .pyc file would
mismatch the installed .py file (which would either be equal to the
timestamp that it was checked out of git, or that it was generated, if
that is the case).