Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#3899 closed (duplicate)

Building RPM fails for 0.96 (python setup.py bdist_rpm)

Reported by: rich@… Owned by: Adrian Holovaty
Component: Core (Other) Version: 0.96
Severity: Keywords: setup
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

There seems to be a mismatch between the version number generated by the setup.py file and the one it should be using. This causes python setup.py bdist_rpm to fail. I've attached a patch that fixes the problem.

Attachments (1)

fix-setup-version-info.patch (122 bytes ) - added by rich@… 17 years ago.
Patch to fix setup bdist_rpm target

Download all attachments as: .zip

Change History (4)

by rich@…, 17 years ago

Patch to fix setup bdist_rpm target

comment:1 by Gary Wilson <gary.wilson@…>, 17 years ago

Resolution: duplicate
Status: newclosed

Duplicate of the outstanding issue of #3338.

comment:2 by Malcolm Tredinnick, 17 years ago

By the way, there are two problems with this patch:

  1. It doesn't tell us which file to patch (although I could work it out, because I know the problem you are trying to solve). In future, please generate patches from the top of the tree so that it includes the full path information. And use the "-u" option to get a unifed diff, because it's a bit shorter in most cases.
  2. It tries to fix the problem by making the real version number a bit less readable (humans are more used to reading dashes in strings than underscores). The fix has to be in the rpm package number generation, not by changing Django's version number to something that reads less like a nomrmal number string. That is why #3338 is still open.

comment:3 by Malcolm Tredinnick, 17 years ago

My apologies... your patch does indeed try to fix the right area if it is meant to applied to setup.py. Still not quite right, though and a dupe of #3338, but I didn't want to leave the impression it was totally ridiculous.

Note: See TracTickets for help on using tickets.
Back to Top