bugWeeChat - Bugs: bug #27933, destdir support fix

 
 

bug #27933: destdir support fix

Submitter:  Antoine Reilles <tonio>
Submitted:  Wed 04 Nov 2009 08:12:19 AM UTC
   
 
Category:  packaging Severity:  3 - Normal
Item Group:  compile/install Status:  Wont Fix
Privacy:  Public Assigned to:  flashcode
Originator Name:  Open/Closed:  Closed
Release:  * 0.3.0 IRC nick: 
* Mandatory Fields

Add a New Comment Rich Markup
   

Sat 17 Dec 2022 12:01:06 PM UTC, comment #3: 

Autotools support removal is scheduled for version 3.9 (CMake will be the only way to build WeeChat).

Thus closing this bug, it will not be fixed.

Sebastien Helleu <flashcode>
Group administrator
Wed 04 Nov 2009 10:18:38 AM UTC, comment #2: 

The patch I provided is wrong: it patches the distributed sources so that weechat-0.3.0 can be installed with destdir, but does not patch the files that is used to generate the Makefile.in file.

Attached patch updates the Makefile.am file, which is used to generate the Makefile.in file in the distribution.

(file #18987)

Antoine Reilles <tonio>
Wed 04 Nov 2009 08:28:20 AM UTC, comment #1: 

Hi Antoine,

In your patch, you modified file Makefile.in in src/plugins but this file it automatically built (it's not in repository). So I can't apply it.

Moreover, cmake is now recommended way to compile weechat, did you try it?
But anyway bugs will still be fixed in configure/Makefiles since they still exist in git repository.

Sebastien Helleu <flashcode>
Group administrator
Wed 04 Nov 2009 08:12:19 AM UTC, original submission:  

Hi,

Destdir is not fully supported by the weechat makefiles, as the install-data-hook rule directly refers to $(includedir).

The fix is simple: prepend $(DESTDIR) in fron of $(includedir) in this rule.
This allows to install weechat in destdir, and ease packaging.

best regards,
antoine

Antoine Reilles <tonio>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #18990:  0001-fix-destdir-support.patch added by tonio (758B - application/octet-stream - git formatted patch)
file #18987:  weechat.diff added by tonio (398B - application/octet-stream - patch for Makefile.am)
file #18986:  patch-ac added by tonio (631B - application/octet-stream - patch proposition)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by flashcode (Posted a comment)
  • -email is unavailable- added by tonio (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-12-17 flashcode StatusNeed Info Wont Fix
        Open/ClosedOpen Closed
    2009-11-04 tonio Attached File- Added 0001-fix-destdir-support.patch, #18990
    2009-11-04 tonio Attached File- Added weechat.diff, #18987
    2009-11-04 flashcode StatusNone Need Info
        Assigned toNone flashcode
    2009-11-04 tonio Attached File- Added patch-ac, #18986

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code