Wireshark

  • Riverbed Technology
  • WinPcap
the world's foremost network protocol analyzer
  • Wireshark
    • About
    • Download
    • Blog
  • Get Help
    • Ask a Question
    • FAQs
    • Documentation
    • Mailing Lists
    • Online Tools
    • Wiki
    • Bug Tracker
  • Develop
    • Get Involved
    • Developer's Guide
    • Browse the Code
    • Latest Builds

Wireshark-bugs: [Wireshark-bugs] [Bug 3120] make rpm-package fails.

Date Index Thread Index Other Months All Mailing Lists
Date Prev Date Next Thread Prev Thread Next


From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Sun, 8 Feb 2009 12:22:58 -0800 (PST)

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3120





--- Comment #4 from Chris Maynard <christopher.maynard@xxxxxxxxx>  2009-02-08 12:22:46 PDT ---
(In reply to comment #3)
> Changed severity back to normal, as it doesn't affect most users (see
> explanation of serverity next to the button). Not being able to build the rpm
> isn't dramatic, you can always install it directly.
You can only install it directly if you can actually compile it on the system
you intend to install it on.  For systems where this is not possible to do,
then I would say this is more than just a "normal" problem; it's actually a
complete show-stopper.

> I'm opposed to both changes:
What exactly is your opposition to the first patch?  If the
packaging/macosx/Info.plist.in isn't copied over, the "make rpm-package" fails,
plain and simple.

> I want things to be installed in /usr instead of /usr/local because I want to
> overwrite existing wireshark installations - troubleshooting installations
> where there are a Distro and a wireshark.org binaries is often a cause for
> problems.
> The "proper" fix for this problem is that *you* set the correct variable in
> *your* rpmmacros file.
You may want things installed in /usr, but if you don't specify a prefix, isn't
the default location actually /usr/local?  The patch simply attempts to
synchronize the wireshark.spec.in file with the default prefix.

> The proper fix here would be a patch that uses the proper macronames in the
> specfile instead of partially hardcoded paths. I'd be perfectly happy to
> integrate such a patch. 
I'm no RPM expert, so I'm not sure what you mean by "proper macronames".  I
merely tried to modify the existing paths to match the defaults.  If you have a
better way to correct this than what I've submitted, then I certainly don't
mind having my patch discarded in favor of a better one.


-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

  • Prev by Date: [Wireshark-bugs] [Bug 3120] make rpm-package fails.
  • Next by Date: [Wireshark-bugs] [Bug 3244] Wireshark doesn't decode CRG message inside APM message in ISUP over M3UA
  • Previous by thread: [Wireshark-bugs] [Bug 3120] make rpm-package fails.
  • Next by thread: [Wireshark-bugs] [Bug 3245] New: It is better for win32-setup.sh --appverify to report all missing files at one time
  • Index(es):
    • Date
    • Thread

Wireshark and the "fin" logo are registered trademarks of the Wireshark Foundation