Bug 224065 - ports-mgmt/poudriere - status page does not work on Safari (HS)
Summary: ports-mgmt/poudriere - status page does not work on Safari (HS)
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Bryan Drewery
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-12-03 22:38 UTC by rainer
Modified: 2017-12-05 19:06 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (bdrewery)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description rainer 2017-12-03 22:38:18 UTC
Hi,


using poudriere 3.22 on FreeBSD 11.1 amd64.

When I go to the status-page, I get the following error in a sort-of modal pop-up:

"
DataTables warning: table id=queued_table - Requested unknown parameter '1' for row 0. For more information about this error, please see http://datatables.net/tn/4
"

This is using Safari Version 11.0.1 (13604.3.5) from High Sierra (macOS 10.13.1).


It works in Firefox 57.1.
Comment 1 SBB 2017-12-04 22:46:43 UTC
I have the same error. Was working fine with poudriere 3.1.19 but now get the same error after upgrading to 3.2.2.

FreeBSD 11.1-RELEASE-p4 amd64
poudriere-3.2.2
Comment 2 Bryan Drewery freebsd_committer freebsd_triage 2017-12-04 23:05:38 UTC
Do you have a URL example I can look at?
Comment 3 SBB 2017-12-04 23:37:15 UTC
Seems clearing the browser cache and restarting fixed it. I'd tried that before and it didn't work but it did work this time.
Comment 4 Bryan Drewery freebsd_committer freebsd_triage 2017-12-04 23:46:13 UTC
(In reply to SBB from comment #3)
> Seems clearing the browser cache and restarting fixed it. I'd tried that
> before and it didn't work but it did work this time.

Yes I think that's what the problem is. The code does not have a sane way
to deal with new unexpected data and instead causes this error.  I will
add an explicit error for when this happens -- it will only happen once
the new assertion is in the cached file for future additions.
Comment 5 commit-hook freebsd_committer freebsd_triage 2017-12-04 23:53:45 UTC
A commit references this bug:

Author: bdrewery
Date: Mon Dec  4 23:53:19 UTC 2017
New revision: 455552
URL: https://svnweb.freebsd.org/changeset/ports/455552

Log:
  Update to 3.2.2-37-gf25a99c8

    - jail -c -m null: Fix crashing with existing-dir error after commit 1d0222b696.
    - jail -c -m src: Respect existing .cpignore properly
    - jail -m null -S <path>: Use the -S path for jail source rather than
      expect <jail>/usr/src.
    - jail -c -m null: Assert that the jail is populated already.
    - HTML: Throw explicit error if unknown ports type is encountered. [1]

  PR:		224065 [1]

Changes:
  head/ports-mgmt/poudriere-devel/Makefile
  head/ports-mgmt/poudriere-devel/distinfo
Comment 6 commit-hook freebsd_committer freebsd_triage 2017-12-05 19:06:46 UTC
A commit references this bug:

Author: bdrewery
Date: Tue Dec  5 19:06:29 UTC 2017
New revision: 455605
URL: https://svnweb.freebsd.org/changeset/ports/455605

Log:
  - Update to 3.2.3

  Changes: https://github.com/freebsd/poudriere/wiki/release_notes_323

     - bulk: Fix MOVED origin not causing a package rebuild.  This
       could result in a reverse dependency continuing to rebuild
       due to a "new dependency" now with FLAVORS support.
     - jail -c -m null: Fix this to work rather than error about
       having an existing directory already.
     - jail -c -m src: Respect existing _/usr/src/.cpignore_
     - jail -c -m null -S: Respect the `-S path` rather than assume the jail has
       a _/usr/src_.
     - jail -c -m null: Assert that the jail is populated already.
     - HTML: Throw explicit error if unknown ports type is encountered. [1]

  PR:	224065 [1]

Changes:
  head/ports-mgmt/poudriere/Makefile
  head/ports-mgmt/poudriere/distinfo