Edgewall Software
Modify

Opened 6 years ago

Closed 5 years ago

#13055 closed task (fixed)

Release Trac 1.0.18

Reported by: Ryan J Ollos Owned by: Ryan J Ollos
Priority: normal Milestone: 1.0.18
Component: general Version:
Severity: normal Keywords: release
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

This ticket is used to coordinate the finalization and testing of the next stable version of Trac, 1.0.18.

Attachments (0)

Change History (21)

comment:1 by Ryan J Ollos, 6 years ago

I'd like to release 1.0.18 after we finish just a few tickets, in order to test deployment to PyPI (#12833).

comment:2 by Ryan J Ollos, 6 years ago

PyPI generates a SHA256 hash (see pypi:Trac/1.2.2/#files), so I propose we generate SHA256 rather than SHA1 for the next release. This will require a minor change to update-latest.sh and the LatestRelease macro (TracDownload).

comment:3 by Ryan J Ollos, 6 years ago

pip 18.0 has been released. Travis CI fails due to use of format=legacy. I'm testing the following change:

  • .travis.yml

    commit 68887f209aedbb382250091ae27ea7739caddc7a (HEAD -> t13055_travis_ci_pip, rjollos/t13055_travis_ci_pip)
    Author: Ryan J Ollos <ryan.j.ollos@gmail.com>
    Date:   Sun Jul 22 17:12:27 2018 -0700
    
        Remove use of unsupported option value
    
    diff --git a/.travis.yml b/.travis.yml
    index 3acf7c701..71c190e3b 100644
    a b before_install:  
    4747    {
    4848      echo "[global]"
    4949      echo "cache-dir = $HOME/.cache/pip"
    50       echo "[list]"
    51       echo "format = legacy"
    5250    } >"$HOME/.pip/pip.conf"
    5351  - |
    5452    if [ "$TRAVIS_OS_NAME" = osx ]; then

comment:4 by Ryan J Ollos, 6 years ago

Better patch, I think:

  • .travis.yml

    commit 5815f7722b05c21921a82473754e0d93170b05ca (HEAD -> t13055_travis_ci_pip)
    Author: Ryan J Ollos <ryan.j.ollos@gmail.com>
    Date:   Sun Jul 22 17:12:27 2018 -0700
    
        Remove use of unsupported option value
    
    diff --git a/.travis.yml b/.travis.yml
    index 3acf7c701..df10618ba 100644
    a b before_install:  
    4848      echo "[global]"
    4949      echo "cache-dir = $HOME/.cache/pip"
    5050      echo "[list]"
    51       echo "format = legacy"
     51      echo "format = columns"
    5252    } >"$HOME/.pip/pip.conf"
    5353  - |
    5454    if [ "$TRAVIS_OS_NAME" = osx ]; then

comment:5 by Ryan J Ollos, 6 years ago

Committed comment:4 patch to 1.0-stable in r16758, merged in r16759, r16760.

in reply to:  2 comment:6 by Ryan J Ollos, 6 years ago

Replying to Ryan J Ollos:

PyPI generates a SHA256 hash (see pypi:Trac/1.2.2/#files), so I propose we generate SHA256 rather than SHA1 for the next release. This will require a minor change to update-latest.sh and the LatestRelease macro (TracDownload).

These changes have been implemented for Trac 1.0.17, 1.2.3 and 1.3.3. See TracDownload.

Last edited 6 years ago by Ryan J Ollos (previous) (diff)

comment:7 by Ryan J Ollos, 6 years ago

The Makefile gives me an error on Windows (using the Make distribution suggested in comment:7:ticket:12630):

>make checksum version=1.2.4.dev0
"Packages for Trac-1.2.4.dev0:"
ECHO is off.
f0376b038b471592b5fdef7e441444f7 *dist/Trac-1.2.4.dev0.win32.exe
Traceback (most recent call last):
  File "contrib/checksum.py", line 38, in <module>
    sys.exit(main() or 0)
  File "contrib/checksum.py", line 29, in main
    with io.open(filename, 'rb') as f:
IOError: [Errno 2] No such file or directory: 'dist/Trac-1.2.4.dev0.win-amd64.exe;'
make: *** [Makefile:662: checksum] Error 1

The following change resolves the issue and doesn't cause a problem for OSX:

  • Makefile

    diff --git a/Makefile b/Makefile
    index b5e011789..6b89de1b4 100644
    a b else  
    681681       @echo "Packages for Trac-$(version):"
    682682       @echo
    683683       @$(if $(packages), \
    684            python contrib/checksum.py md5:sha1 $(packages); \
     684           python contrib/checksum.py md5:sha1 $(packages) \
    685685       , \
    686686           echo "No packages found: $(sdist+wheel) $(wininst)" \
    687687       )

comment:8 by Ryan J Ollos, 6 years ago

Modified make checksum to address comment:6 and comment:7 in r16787, merged in r16788 and r16789.

comment:9 by Ryan J Ollos, 6 years ago

I've tried using different distributions of make, including the suggestion in comment:7:ticket:12630. More recently, I've installed make on Windows 10 via choco install make. In each case, make checksum version=... and make release succeed, but make upload version=... fails:

> make upload version=1.2.4.dev0
"Packages for Trac-1.2.4.dev0:"
ECHO is off.
89f5ac66907d3b55c211a83d343c5a4d *dist/Trac-1.2.4.dev0.win32.exe
5c508f61d6131642ffc747bb7e2c68a2 *dist/Trac-1.2.4.dev0.win-amd64.exe
08b4645f4f80a9dc1e5a93ac52275116293d3692884a9f4786d72778bd02cd53 *dist/Trac-1.2.4.dev0.win32.exe
23d2789846379661460f3fd5580f488c0bd10b7e74c1f4682f3ea5c5e33881ce *dist/Trac-1.2.4.dev0.win-amd64.exe
C:/WINDOWS/System32/OpenSSH/scp.exe -i C:/Users/rjollos/Documents/Workspace/trac-dev/id_rsa dist/Trac-1.2.4.dev0.win32.exe dist/Trac-1.2.4.dev0.win-amd64.exe rjollos@lynx.edgewall.com:/home/rjollos/dist
process_begin: CreateProcess(NULL, C:/WINDOWS/System32/OpenSSH/scp.exe -i C:/Users/rjollos/Documents/Workspace/trac-dev/id_rsa dist/Trac-1.2.4.dev0.win32.exe dist/Trac-1.2.4.dev0.win-amd64.exe rjollos@lynx.edgewall.com:/home/rjollos/dist, ...) failed.
make (e=2): The system cannot find the file specified.
make: *** [upload] Error 2

The strange thing is that it works to run the same command in the shell: C:/WINDOWS/System32/OpenSSH/scp.exe -i C:/Users/rjollos/Documents/Workspace/trac-dev/id_rsa dist/Trac-1.2.4.dev0.win32.exe dist/Trac-1.2.4.dev0.win-amd64.exe rjollos@lynx.edgewall.com:/home/rjollos/dist.

So there is some syntax or other problem with running that command from the Makefile.

comment:10 by Jun Omae, 6 years ago

I have the same issue on Windows 10 with C:/WINDOWS/System32/OpenSSH/scp.exe. Instead, pscp.exe (scp of PuTTY) works fine.

comment:11 by Ryan J Ollos, 6 years ago

PuTTY works for me too. Thanks!

Documented in TracDev/ReleaseChecklist@131.

comment:12 by Ryan J Ollos, 6 years ago

Added new update-help Makefile rule in [16796:16798]. Revised TracDev/ReleaseChecklist@133.

in reply to:  12 comment:13 by Jun Omae, 6 years ago

Replying to Ryan J Ollos:

Added new update-help Makefile rule in [16796:16798]. Revised TracDev/ReleaseChecklist@133.

$(PYTHON) variable is introduced for Python interpreter in r14624 for 1.2-stable. We could use the variable in 1.2-stable and trunk.

        @echo "Packages for Trac-$(version):"
        @echo
        @$(if $(packages), \
-           python contrib/checksum.py md5:sha256 $(packages) \
+           $(PYTHON) contrib/checksum.py md5:sha256 $(packages) \
        , \
            echo "No packages found: $(sdist+wheel) $(wininst)" \
        )
 endif

 update-help:
-       @python contrib/checkwiki.py -d --prefix=$(prefix)
-       @python contrib/wiki2rst.py TracUpgrade > UPGRADE.rst
-       @python contrib/wiki2rst.py TracInstall > INSTALL.rst
+       @$(PYTHON) contrib/checkwiki.py -d --prefix=$(prefix)
+       @$(PYTHON) contrib/wiki2rst.py TracUpgrade > UPGRADE.rst
+       @$(PYTHON) contrib/wiki2rst.py TracInstall > INSTALL.rst

comment:14 by Ryan J Ollos, 6 years ago

Refactoring in r16799.

comment:13 changes committed in r16800, r16803.

comment:15 by Ryan J Ollos, 5 years ago

I'm aiming to complete #13087 and release on Saturday 02/16.

in reply to:  15 comment:16 by Ryan J Ollos, 5 years ago

Replying to Ryan J Ollos:

I'm aiming to complete #13087 and release on Saturday 02/16.

Missed on that date, but aiming to complete this coming weekend.

comment:17 by Ryan J Ollos, 5 years ago

I'm finding some more free time now, so if we can get all the remaining tickets closed out I will try to get the releases done.

comment:18 by Ryan J Ollos, 5 years ago

I was going to release today, but #13087 needs to be resolved. I'll post proposed changes for that ticket today and plan to release next weekend. Please try to commit any proposed changes by end of week.

comment:19 by Ryan J Ollos, 5 years ago

I'll make the release on Friday 7/12, or Sunday 7/14 at the latest.

comment:20 by Ryan J Ollos, 5 years ago

Owner: set to Ryan J Ollos
Status: newassigned

comment:21 by Ryan J Ollos, 5 years ago

Resolution: fixed
Status: assignedclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Ryan J Ollos to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.